Skip to content

don't allow uris inside ${ ... } blocks#1019

Closed
fkz wants to merge 1 commit intoNixOS:masterfrom
fkz:antiquotes
Closed

don't allow uris inside ${ ... } blocks#1019
fkz wants to merge 1 commit intoNixOS:masterfrom
fkz:antiquotes

Conversation

@fkz
Copy link
Contributor

@fkz fkz commented Aug 9, 2016

this fixes #1017
also related to #836
evaluating nix-env -qa worked (modulo some typo problem) so this doesn't seem to be used inside nixpkgs.
this is a non-backward-compatible change though, 3rd party code could break because of this.

this fixes NixOS#1017
evaluating `nix-env -qa` and (modulo some typo problem) this doesn't seem to be used inside nixpkgs.
this is a non-backward-compatible change though, 3rd party code could break because of this.
@domenkozar
Copy link
Member

I think it's better here to just not allow all RFC urls, as this error also happens in functions such as a:b:c:.

@stale
Copy link

stale bot commented Feb 13, 2021

I marked this as stale due to inactivity. → More info

@stale stale bot added the stale label Feb 13, 2021
@stale
Copy link

stale bot commented Apr 16, 2022

I closed this issue due to inactivity. → More info

@stale stale bot closed this Apr 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Antiquotes silently fail

4 participants