-
Notifications
You must be signed in to change notification settings - Fork 0
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
Draft: 7 add rest service to ingestor project #16
Conversation
* Add repo skeleton and desktop-app framework Add wails.io based app with svelte frontend, a taskqueue and basic go module and package structure * Follow standard go project structure * Add CI workflow * App: Don't fail when config is missing * Add linting * Fix defer * Abstract progress notifier Removes dependency of taskqueue on wails * Move core package into separate folder
internal/webserver/api.gen.go
Outdated
router.GET(options.BaseURL+"/version", wrapper.OtherControllerGetVersion) | ||
} | ||
|
||
// Base64 encoded, gzipped, json marshaled Swagger object |
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.
That's one way to do it. Weird that oapi-codegen embeds the schema instead of referencing the schema file directly.
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.
¯\(ツ)/¯
What are your first impressions? Do you foresee any issues with the oapi-codegen+gin stack after the initial setup? The code smells like generated code, but otherwise looks fine to me at a first glance. |
Disable bearer auth for development
1bc89c1
to
c485052
Compare
Generating code works fine, all the middleware (auth) can be configured. The only thing I struggled with was creating a Swagger UI test page (see latest commits) This needs to be annotated manually which I find really odd given that we have openapi specs already. Either I'm doing it wrong or that's just the way it is. References: |
Create docs from function annoations to be served under /docs/index.html
87a47b1
to
2e4fe76
Compare
2e4fe76
to
4a51612
Compare
795f91b
to
893258c
Compare
@consolethinks Should we merge this? |
No description provided.