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

web server instead of batch report generation #15

Merged
merged 15 commits into from
Dec 11, 2019
Merged

web server instead of batch report generation #15

merged 15 commits into from
Dec 11, 2019

Conversation

bzz
Copy link
Owner

@bzz bzz commented Dec 6, 2019

Addresses #14

TODOs:

  • separate server binary (easier that way, offline/online could be united to single binary later)
  • 3-legged Oauth2.0 flow support
  • multitenancy: store session tokens per-user (in cookies)
  • multitenancy: allow to choose a label name, store it per-user
  • aggregation & HTML rendering of messages
  • better abstract split (unicode friendly, whitespace-aware)
  • automatic token refresh automation will be added later,
    it's enough that it redirects to /login on token expiration now
  • add API fetch progress indicator
  • main page rendering using html/template

@bzz bzz changed the title WIP: web server WIP: the web server Dec 6, 2019
@bzz
Copy link
Owner Author

bzz commented Dec 8, 2019

Current implementation is working 🎉
it just does include the report template yet.

It's a tad slow to be interactive, as one need to fetch every every message in individual API call (until googleapis/google-api-go-client#435), so added another item about visual progress indicator for a better user feedback.

@bzz bzz changed the title WIP: the web server WIP: web server instead of batch report generation Dec 8, 2019
@bzz
Copy link
Owner Author

bzz commented Dec 9, 2019

Screen Shot 2019-12-09 at 5 02 05 PM

@bzz
Copy link
Owner Author

bzz commented Dec 10, 2019

Initial version of the server is working, going to polish just a few be bits and merge it asap as it includes useful refactoring/improvements to the CLI as well.

Further usability improvements (message loading time, better template rendering, etc) will be handled under a separate issues.

@bzz bzz changed the title WIP: web server instead of batch report generation web server instead of batch report generation Dec 11, 2019
@bzz
Copy link
Owner Author

bzz commented Dec 11, 2019

Rebased on latest master & ready to :shipit:

@bzz bzz merged commit a0902f0 into master Dec 11, 2019
@bzz bzz deleted the add-server branch December 11, 2019 21:39
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.

1 participant