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
HTTPS urls #60010
HTTPS urls #60010
Conversation
Lots of URLs were HTTP redirect to HTTPS. Changed those and checked them if there's actual content. Inspired by NixOS#60004
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. I tried a bunch of random links and they worked fine. Thanks!
If found pretty much all of those because repology said they automatically redirect to HTTPS when you try to access HTTP: https://repology.org/repository/nix_unstable/problems And almost all of them are the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me. Moving fetchurl to https is not as important because we have hashes to ensure integrity, but it also doesn't hurt.
@matthiasbeyer It's important for whenever somebody updates the hashes |
@infinisil you wanted @matthewbauer probably. |
sha256 = "022phbrsny3gb8npb6sxyqqxacx138q5bd7dq3gqxh3kprx5chbl"; | ||
}) | ||
]; | ||
|
||
propagatedBuildInputs = [ zest ]; | ||
|
||
meta = with stdenv.lib; { | ||
homepage = http://www2.cs.tum.edu/projects/cup/eclipse.php; | ||
homepage = https://www2.cs.tum.edu/projects/cup/eclipse.php; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
link is dead
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hmmm, how did I miss that? Seems like they support HTTPS but have different content there.
a6cf2a4
to
92cccb6
Compare
Motivation for this change
Lots of URLs were HTTP redirect to HTTPS. Changed those and checked them
if there's actual content. Inspired by #60004
They're mostly homepage URLs so there are probably a lot of source URLs left on HTTP which we should fix.
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)