-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Root cert not installed in local store with localias start
#30
Comments
Interesting, thanks for reporting the issue, and sorry for the trouble. It's quite likely there is an issue with localias, so I'd like to be able to fix it. Could you please help me by providing a little more information?
Again, thanks for taking the time to report the problem, all these additional questions will just help me fix it faster. |
No problem. I didn't know where to start as far as what you wanted.
I'm having a hard time reproducing. I'd have to figure out how to remove the root cert from the local store I think. |
If it helps. There was a difference for us when running |
Interesting, thanks for the more information. I'm taking a cursory look now but I will keep investigating as I have time over the rest of the week. Thank you again for your helpful replies. I have a few more followups:
How did you know that the root cert was not installed in "the local store"? I'd like to know what behavior you expected, and what behavior you observed. If the behavior involves a browser, please mention which one.
I'm not sure exactly what you mean by "the local store", but you can remove entries from the system keychain by opening "Keychain Access.app" (search for "keychain access" with spotlight, or it should be in If you're using Firefox, you can check the Certificates manager by opening Preferences and then searching "cert" and clicking the "View Certificates" button. From there, find any localias-related certs in the Authorities tab and in the Your Certificates tabs and delete them.
I'm on an older version of macOS but I cannot reproduce the behavior you've described about a difference between |
This is very detailed thanks. I do not get this behavior though, nor did the other 2 people on the team. When we run
The cert wasn't trusted when we'd use our locally running server in chrome or safari. Based on the README, I expected the root cert to be installed and trusted in chrome. I applaud how responsive you've been and if you really want to try to get to the bottom of my particular environment (and my teammates') I'd be happy to do a screenshare sometime with you so we can try to debug it. Just let me know :) (edit: I'll also add that I'm happy with how I at least got it working. At a minimum maybe this helps you regarding any notes or FAQ you might want to add to the README.) |
OK, I appreciate the help and I'm sorry that I can't reproduce the issue. I won't have time to upgrade my computer for a while, so until then I don't think there's much I can do. If you can write back in with a reliable way to reproduce the issue, and clearer details on what exactly you and your team are observing (are you all on the same OS version? are there any similarities or differences in your environments? etc.) and which expectations are violated at which step, I'd be happy to pick this back up. I'll also happily accept a PR if you can fix the issue on your own by modifying the source! Again, thank you for your patience and I'm sorry I can't fix it. I'm glad that 🫡 Sincerely, |
I don't have anything too useful to add except that I also saw this behavior. I initially chalked it up to my user error though until I saw this issue. 😅
I'm also on sonoma, so maybe it's something related to that. 🤔 |
OK, I've finally had time to upgrade my OS to Sonoma (14.4.1) and I have been able to reproduce the problem! I'm going to fix this by implementing daemonization in a slightly different way — instead of starting the Caddy server in the background/daemonized process, I'm going to start it and only then fork the daemon process. In some local testing, this seems to fix the problem. PR to come shortly, release shortly after that. Thank you all for your patience and your help. |
I'm very sure that the linked pr ^ will fix the issue (because I was able to repro), and I plan on merging and releasing a new version within the next week or so. If you'd like to sneak onto the next version before release, feel free to check out the PR and build a binary yourself! That branch also includes some nice quality of life changes, like |
…#31) ## Bugfix Fix #30 by changing how daemonization works. - Previously: `start`/`reload` would apply the latest config in the foreground process, then fork(), then start Caddy. - Now: `start`/`reload` apply the latest config, starts Caddy in the foreground process, then fork()s. - By starting Caddy in the foreground process, any activities requiring sudo/root will happen while the program is still interactive. - This fixes the problem (adding the root cert to the system store requires sudo/root privileges, but because Caddy was running in a background process, it couldn't request that permission.) - The foreground process exits cleanly and the background process starts Caddy again, this time without any need for sudo/root privileges because any of the work it needed to do was already done in the foreground process. ## Friendlier CLI This change encouraged me to make the following improvements to the CLI commands: - `stop` just kills the daemon, if it's running. If the daemon wasn't running, `stop` will now exit cleanly. Previously, it would throw an error, but that's user-hostile because the goal of running `stop` is to ensure no running daemon — if that goal is accomplished by stopping a running daemon, or confirming no daemon was running, doesn't really matter. - `start` will start a new daemon. If one was already running, it will be killed and replaced by a new one. The user's goal is to ensure a daemon is running, with the latest config, and this is achieved regardless of whether or not there was an existing daemon. - `reload` becomes an alias for `start`, because they have the exact same behavior — ensure that a daemon is running with the latest configuiration. ## Get rid of `caddymodules` A while ago, localias was built using `gomod2nix`, and there was an incompatibility between that helper and the opentelemetry modules included in Caddy. To work around this, I created a `caddymodules` package that imported all of the Caddy modules _except_ opentelemetry, which was fine because this project doesn't use the opentelemetry modules in any way. Because localias no longer uses `gomod2nix`, this PR gets rid of the `caddymodules` hack entirely. This then allowed me to upgrade the version of Caddy that is being installed, and it will make it easier to stay up to date as Caddy receives further improvements. ## SSL renewal server With an upgraded Caddy came a problem — for SSL issuance, Caddy now requires you to implement an "automation policy" server that confirms that it can issue a new certificate for a given domain. This is primarily aimed at issuing certificates for real life domains accessible to the public, not for internal development aliases, but the restriction still stands. To do this, I used Caddy itself to respond to these requests. For more information, read: - https://caddyserver.com/docs/automatic-https#on-demand-tls - https://caddy.community/t/serving-tens-of-thousands-of-domains-over-https-with-caddy/11179 - caddyserver/caddy#6055 In the future, I could implement this by writing a custom policy module instead of using the HTTP ask, but this works for now. ## Dependencies cleanup - General updates of all imported golang packages. - Update the `flake.nix` and `flake.lock` files, switch to `buildGoModule` instead of `buildGo120Module` to make it easier to use this flake with an override `nixpkgs` upstream.
@twalling @ericf89 should be fixed now — if you installed via brew, you can |
Happy to provide more context here but we found that unless we ran
localias run
, we wouldn't get the root cert installed in the local store. Running it as a daemon did prompt for sudo but for some reason it wasn't the same as running it in the foreground. I wanted to report it in case other people ran into it, or if the documentation should be updated or if it's something that can actually be worked around.The text was updated successfully, but these errors were encountered: