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

Change http to https for security links #826

Merged
merged 1 commit into from
Feb 21, 2019
Merged

Change http to https for security links #826

merged 1 commit into from
Feb 21, 2019

Conversation

huynguyennovem
Copy link

Please fill in the fields below to submit a pull request. The more information
that is provided, the better.

Fixes issue #: N/A

Description of the changes being introduced by the pull request:
For security, we should change http into https links.

Please verify and check that the pull request fulfills the following
requirements
:
N/A

For security, we should change http into https links.

Signed-off-by: huynq0911 <huynq0911@gmail.com>
Copy link
Member

@JustinCappos JustinCappos left a comment

Choose a reason for hiding this comment

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

LGTM

@JustinCappos JustinCappos merged commit 9f5c401 into theupdateframework:develop Feb 21, 2019
Copy link
Member

@lukpueh lukpueh left a comment

Choose a reason for hiding this comment

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

Thanks for the PR, @huynq0911. Have you tried any of the links? Some of them don't seem to work (see my comments inline). I vote for reverting this merge. We should seize the opportunity and go through the files in docs to update outdated links.

@@ -1,7 +1,7 @@
## Adopters
* [Advanced Telematics Systems](https://advancedtelematic.com/en/press-releases/ats-is-integrating-the-uptane-security-framework-for-over-the-air-software-updates-to-connected-vehicles.html)
* [Airbiquity](https://www.airbiquity.com/news/press-releases/airbiquity-unveils-new-otamatictm-release-further-strengthening-over-air-ota-software-and-data-management-offering-automotive)
* [Automotive Grade Linux](http://docs.automotivelinux.org/docs/architecture/en/dev/reference/security/05-platform-security.html#agl-platform-software-update)
* [Automotive Grade Linux](https://docs.automotivelinux.org/docs/architecture/en/dev/reference/security/05-platform-security.html#agl-platform-software-update)
Copy link
Member

Choose a reason for hiding this comment

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

This link seems dead (at least the path).

Copy link
Author

Choose a reason for hiding this comment

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

@@ -4,7 +4,7 @@ Metadata files provide information that clients can use to make update decisions

The signed metadata files always include the time they were created and their expiration dates. This ensures that outdated metadata will be detected and that clients can refuse to accept metadata older than that which they've already seen.

All TUF metadata uses a subset of the JSON object format. When calculating the digest of an object, we use the [Canonical JSON](http://wiki.laptop.org/go/Canonical_JSON) format. Implementation-level detail about the metadata can be found in the [spec](docs/tuf-spec.txt).
All TUF metadata uses a subset of the JSON object format. When calculating the digest of an object, we use the [Canonical JSON](https://wiki.laptop.org/go/Canonical_JSON) format. Implementation-level detail about the metadata can be found in the [spec](docs/tuf-spec.txt).
Copy link
Member

Choose a reason for hiding this comment

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

This server does not support https.

Copy link
Author

Choose a reason for hiding this comment

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

I have just reverted this to http

@@ -1,4 +1,4 @@
# Tox (http://tox.testrun.org/) is a tool for running tests
# Tox (https://tox.testrun.org/) is a tool for running tests
Copy link
Member

Choose a reason for hiding this comment

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

The subdomain link is dead and the organizational domain seems to be something else. Maybe this should point to https://tox.readthedocs.io/en/latest/ ?

Copy link
Author

Choose a reason for hiding this comment

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

Yes, I think so

@huynguyennovem
Copy link
Author

Thanks for the PR, @huynq0911. Have you tried any of the links? Some of them don't seem to work (see my comments inline). I vote for reverting this merge. We should seize the opportunity and go through the files in docs to update outdated links.

Thank for your review @lukpueh , I just updated by your comment and I create a new PR #828 to update this. Please check it!

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