Skip to content
This repository has been archived by the owner on Jun 17, 2023. It is now read-only.

Oauth2 Integration with Routes #54

Open
wants to merge 39 commits into
base: main
Choose a base branch
from

Conversation

cool-pants
Copy link
Member

  • Added NGINX Configuration
  • Added Prometheus conf for site statistics
  • Abstracted endpoints and methods
  • Added Oauth2 settings

@aylmerbritto
Copy link

Folks,
The implementation is impressive and I just love the structure how everything is organised. It is par with any industrial code.
I had a couple of concerns which any of you can help me understand:

I see we have imported the logging module but I don't see any log levels inside any routes/docs. Is logging levels yet to implemented or are we auto-configuring it somewhere that I am not aware of ?
Prometheus: what is the lowest level that we are tracking it to ? It stops in the route level or does it penetrate to the utility functions too ?

@AbhijithGanesh
Copy link
Member

@aylmerbritto
I would love to reply to all your questions but I prefer it to be done through a meeting

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants