Skip to content
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

Use HTTPS for downloading binaries #16427

Closed
wants to merge 2 commits into from
Closed

Conversation

brson
Copy link
Contributor

@brson brson commented Aug 11, 2014

No description provided.

@brson
Copy link
Contributor Author

brson commented Aug 11, 2014

The first commit changes the docs to use the static.rust-lang.org address for rustup.sh.

@steveklabnik
Copy link
Member

woo!

@brson
Copy link
Contributor Author

brson commented Aug 11, 2014

Hm, firefox isn't quite happy with our validation: https://static.rust-lang.org/doc/master/

@brson
Copy link
Contributor Author

brson commented Aug 11, 2014

I'm told the validation is fine. Firefox doesn't validate our 'identity' because we don't have an 'EV' cert. The mixed content can or can't be addressed later.

@steveklabnik
Copy link
Member

bors added a commit that referenced this pull request Aug 11, 2014
@bors bors closed this Aug 12, 2014
lnicola pushed a commit to lnicola/rust that referenced this pull request Jan 28, 2024
…tics, r=Veykril

fix: filter out cfg disabled filed when lowering `RecordPat`

we should filter out field with disabled cfg when lowering ast `RecordPat` to hir.

close rust-lang/rust-analyzer#16169
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants