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

Is the version unambiguous? #14

Open
chadwhitacre opened this issue May 4, 2016 · 4 comments
Open

Is the version unambiguous? #14

chadwhitacre opened this issue May 4, 2016 · 4 comments

Comments

@chadwhitacre
Copy link

Is the Fernet spec's versioning unambiguous? The spec on current master says:

This document describes version 0x80 (currently the only version) of the fernet format.

However, the current master is the sixteenth commit since the doc started, and it was called 0x80 when it started, too. What did those commits change? Should some of them have bumped the version?

The current lack of tags further suggests that version 0x80 is ambiguous.

@chadwhitacre chadwhitacre changed the title is the version unambiguous? Is the version unambiguous? May 4, 2016
@chadwhitacre
Copy link
Author

From a skim of the 16 commits, here are the ones that jump out as potentially version-bumping changes:

ca8b345 pins to CBC, meaning that implementations of 1fa950a's 0x80 don't guarantee CBC.

58d2827 appears to change from nanoseconds to seconds—the commit message ("new format") and diff (wait—"gAAAAA" is new?! 😳 ) in 115850d further suggest a version-bump-worthy change.

3bbe466 "document checking version byte"—so it is new

c55d0d4 diff suggests that base64url was indicated in at least some specific locations in the original 0x80—was it so indicated everywhere?

0514d7d clear algorithm change

was it so indicated everywhere?

0250c59 Answer: no.

@chadwhitacre
Copy link
Author

chadwhitacre commented May 4, 2016

Is the version unambiguous?

The answer appears to be "yes."

In which case, the action item here is to ... what? Clean up Fernet's versioning somehow. By applying new version numbers to the version-bumping commits above? By dropping back to Git SHA for version? By adopting semantic versioning?

@chadwhitacre
Copy link
Author

To be explicit: this issues makes auditing somewhat more difficult.

@chadwhitacre
Copy link
Author

Clean up Fernet's versioning somehow.

By tagging 0250c59 as 0x80 and adopting a versioning policy for future changes?

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

No branches or pull requests

1 participant