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
nixos/httpd: respect hostName when provisioning certificates #96923
Conversation
ping @arianvp I haven't looked but does |
@mohe2015 please follow CONTRIBUTING.md for the commit message: |
ebf0c12
to
d2d883b
Compare
Done. Hope this is fine now. |
@mohe2015 great! Thanks! |
Maybe don't merge in favor of #91121? |
I'm not pushed honestly. If this is merged I can do a rebase in #91121 this evening. |
It's up to you people - the change in https://github.com/NixOS/nixpkgs/pull/91121/files#diff-67cf2394795df903410ca0d6de1fa388R658 is a little bit different but I don't know what's the actual difference. I just don't want to prevent getting #91121 merged quickly. But it seems like it may not make it into 20.09 so getting this in may be benefical. |
I looked at this a day ago and I think the only difference is that I used builtins.map instead of mapAttrs', as to match the nginx code. The functionality is the same. |
Congrats for getting the other PR merged! |
Thanks Moritz :D
…On Mon 7 Sep 2020, 12:44 a.m. Moritz Hedtke, ***@***.***> wrote:
Congrats for getting the other or merged!
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
<#96923 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAXHIVWY4QQZNCWQTM66X2LSEQNELANCNFSM4QSFHGKQ>
.
|
Motivation for this change
This fixes an inconsistency when the attribute name doesn't match the hostName.
See #96910 (comment)
@aanderse
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)