-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[FEATURE] Add API docs and swagger-ui #1544
Conversation
ArtifactsThese changes are published for testing on Buildkite and DockerHub. Docker Container
|
ccfc7ae
to
2e3e484
Compare
Codecov Report
@@ Coverage Diff @@
## master #1544 +/- ##
==========================================
+ Coverage 66.07% 66.08% +0.01%
==========================================
Files 144 144
Lines 4015 4028 +13
Branches 168 168
==========================================
+ Hits 2653 2662 +9
- Misses 1143 1148 +5
+ Partials 219 218 -1
Flags with carried forward coverage won't be shown. Click here to find out more.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
99a77e6
to
5825019
Compare
fa28652
to
bf12954
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This change will serve out swagger-ui at the `/api/` root path.
This change also factorises and renames index.go into template.go, this can now be generically utilised to template any file.
7475aef
to
977137f
Compare
This change is to more closely conform to the golang-standards project layout.
977137f
to
4aa1408
Compare
This change will serve out swagger-ui at the
/api/
path.I have not documented the following endpoints:
/api/secondfactor/u2f/sign_request
/api/secondfactor/u2f/sign
@clems4ever I'm hoping that you might be able to add these two into the API spec since you understand the flow much better and can actually test inputs/outputs with a u2f key.
Some of the summaries and descriptions probably need some work, happy for a review and any recommendations.
EDIT: I've given all of the summaries and descriptions another review and made some changes.
Closes #450.