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

oauthdevice: Support different auth flows based on server version #27

Closed
wants to merge 2 commits into from

Conversation

minor-fixes
Copy link
Contributor

No description provided.

Currently, version info is available as part of the version information
returned pre-formatted for printing to a console/log. However, we want
to get version info to share with an external server, so this change
creates such a function.

If the binary has no stamped version info, the function will error
(which implies that during development, testing against a real server
will require version stamping to be enabled).

If the released binary is a real release, it will return a common
version string (`vX.Y.Z`). If the binary is not a real release, it will
return a pseudo-version similar to the format used for [Go
modules](https://go.dev/ref/mod#pseudo-versions) - namely,
`v0.0.0-yyyymmddhhmmss-abcdefabcdef`, where the date is the build date,
and the commit is the commit the tool's build was based on. No component
of the version string will indicate if the code tree was dirty.

Tested: unit tests only
Bug: linear/CUS-385
@minor-fixes minor-fixes force-pushed the scott/CUS_385 branch 2 times, most recently from 6d679e5 to 1e83633 Compare August 14, 2024 20:44
Base automatically changed from scott/CUS_385 to main August 14, 2024 20:49
@jayconrod jayconrod closed this Nov 5, 2024
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.

2 participants