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

Update HTTP->HTTPS when both URLs work #1714

Merged
merged 3 commits into from Mar 9, 2019

Conversation

fulldecent
Copy link
Contributor

This updates each HTTP links to HTTPS if and only if the updated URL is accessible.

We should update our links to HTTPS because this promotes privacy.

This is a pure-play commit for the above feature and it does not touch anything else.

One reason is that there are so many HTTP URLs is because for a brief time the eips.ethereum.org website was available on HTTP without redirect. Therefore, after this change, I expect in the future there will be much less need for changes like the instant PR.

@eip-automerger
Copy link

eip-automerger commented Jan 20, 2019

Hi! I'm a bot, and I wanted to automerge your PR, but couldn't because of the following issue(s):

Copy link
Contributor

@yoav-tabookey yoav-tabookey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1613: https://bit.ly/2EWWVN8 is fine.

EIPS/eip-1.md Show resolved Hide resolved
@@ -28,16 +28,16 @@ Moving to PoS has been on the roadmap since day 0 for ethereum, along with a red

#### Asics and advantadges of PoW

[EIP 960](http://eips.ethereum.org/EIPS/eip-969) proposes a change in algorithm to avoid mining being dominated by ASICS. Counter arguments by Phil Daian argue among others than [resisting economies of scale is futile and there might be specific security advantadges to specialized hardware](https://pdaian.com/blog/anti-asic-forks-considered-harmful/). One of the main arguments for PoW mining, even when it doesn't provide security, it is useful as a fair distribution mechanism, that **PoW allows any person with a computer, internet access and electricity to obtain currency without having to deal with government imposed currency controls**.
[EIP 960](https://eips.ethereum.org/EIPS/eip-969) proposes a change in algorithm to avoid mining being dominated by ASICS. Counter arguments by Phil Daian argue among others than [resisting economies of scale is futile and there might be specific security advantadges to specialized hardware](https://pdaian.com/blog/anti-asic-forks-considered-harmful/). One of the main arguments for PoW mining, even when it doesn't provide security, it is useful as a fair distribution mechanism, that **PoW allows any person with a computer, internet access and electricity to obtain currency without having to deal with government imposed currency controls**.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
[EIP 960](https://eips.ethereum.org/EIPS/eip-969) proposes a change in algorithm to avoid mining being dominated by ASICS. Counter arguments by Phil Daian argue among others than [resisting economies of scale is futile and there might be specific security advantadges to specialized hardware](https://pdaian.com/blog/anti-asic-forks-considered-harmful/). One of the main arguments for PoW mining, even when it doesn't provide security, it is useful as a fair distribution mechanism, that **PoW allows any person with a computer, internet access and electricity to obtain currency without having to deal with government imposed currency controls**.
[EIP 960](https://eips.ethereum.org/EIPS/eip-969) proposes a change in algorithm to avoid mining being dominated by ASICS. Counter arguments by Phil Daian argue among others that [resisting economies of scale is futile and there might be specific security advantadges to specialized hardware](https://pdaian.com/blog/anti-asic-forks-considered-harmful/). One of the main arguments for PoW mining, even when it doesn't provide security, it is useful as a fair distribution mechanism, that **PoW allows any person with a computer, internet access and electricity to obtain currency without having to deal with government imposed currency controls**.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not going to look at the rest of this.

@expede
Copy link
Contributor

expede commented Jan 21, 2019

EIP 1066 requires approval from one of (@carchrae, @naumenkogs, @expede)
[...]
EIP 1444 requires approval from one of (@expede, @jenncoop)

I don't have official approval rights on this repo, but the changes to 1066 & 1444 look good 👍

@ligi
Copy link
Member

ligi commented Jan 22, 2019

I don't have official approval rights on this repo, but the changes to 1066 & 1444 look good +1

perhaps it would make sense to modify this bot so a thumbs up also counts as approval if the person does not have the permission to approve

@Arachnid
Copy link
Contributor

There's no "approval rights"; anyone can approve a PR. Just go to the code tab, click "Review Changes", "Approve", and "Submit".

@expede
Copy link
Contributor

expede commented Jan 25, 2019

Indeed. Welp, that's embarrassing! 😅🤣

@nicksavers
Copy link
Contributor

I agree with this PR but currently has conflicts. @fulldecent

@fulldecent
Copy link
Contributor Author

@nicksavers thank you, de-conflicted

@nicksavers nicksavers merged commit 035a021 into ethereum:master Mar 9, 2019
@nicksavers
Copy link
Contributor

I verified that all the updated links are working. Thanks for the work @fulldecent

@fulldecent fulldecent deleted the feature-https branch March 12, 2019 03:59
@fulldecent
Copy link
Contributor Author

Thanks for the review!

ilanolkies pushed a commit to ilanolkies/EIPs that referenced this pull request Nov 12, 2019
@Dukdig2325
Copy link

This updates each HTTP links to HTTPS if and only if the updated URL is accessible.

We should update our links to HTTPS because this promotes privacy.

This is a pure-play commit for the above feature and it does not touch anything else.

One reason is that there are so many HTTP URLs is because for a brief time the eips.ethereum.org website was available on HTTP without redirect. Therefore, after this change, I expect in the future there will be much less need for changes like the instant PR.

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.

None yet

10 participants