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
Improve hash parsing and errors #3730
Merged
edolstra
merged 43 commits into
NixOS:master
from
obsidiansystems:better-ca-parse-errors
Aug 5, 2020
Merged
Improve hash parsing and errors #3730
edolstra
merged 43 commits into
NixOS:master
from
obsidiansystems:better-ca-parse-errors
Aug 5, 2020
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- Ensure hash is in form <algo>-<prefix> and not SRI. - Better errors if something goes wrong - string_view for no coppying
Ericson2314
changed the title
Improve content address parsing
Improve content address parsing and errors
Jun 22, 2020
… into better-ca-parse-errors
Ericson2314
changed the title
Improve content address parsing and errors
WIP: Improve content address parsing and errors -- contains #3738
Jun 29, 2020
WIP because will also fix some |
Ericson2314
changed the title
WIP: Improve content address parsing and errors -- contains #3738
Improve content address parsing and errors -- contains #3738
Jul 2, 2020
Ericson2314
changed the title
Improve content address parsing and errors -- contains #3738
Improve hash parsing and errors -- contains #3738
Jul 2, 2020
Ericson2314
force-pushed
the
better-ca-parse-errors
branch
from
July 2, 2020 21:48
777dbb5
to
2f93d9f
Compare
… into better-ca-parse-errors
Ericson2314
changed the title
WIP: Improve hash parsing and errors -- contains #3738
Semi-WIP: Improve hash parsing and errors -- contains #3738
Jul 16, 2020
Ericson2314
changed the title
Semi-WIP: Improve hash parsing and errors -- contains #3738
Improve hash parsing and errors -- contains #3738
Jul 16, 2020
… into better-ca-parse-errors
…ix into better-ca-parse-errors
edolstra
reviewed
Jul 20, 2020
edolstra
reviewed
Jul 20, 2020
OK, fixed both. |
Fixed some more conflicts |
Ah, it is already fixed on this branch. |
… into better-ca-parse-errors
Conflicts fixed. |
@edolstra wiggled the metadata so GitHub would recompute the diff, and this PR is actually readable. |
OK this should be ready. |
Ericson2314
changed the title
Improve hash parsing and errors -- contains #3738
Improve hash parsing and errors
Jul 31, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Ensure hash is in form
<algo>-<prefix>
and not SRI.Better errors if something goes wrong
std::string_view
for no coppying