Skip to content

Add query endpoint for trusted users - #97

Open
MialLewis wants to merge 24 commits into
mainfrom
add_query_endpoint
Open

Add query endpoint for trusted users#97
MialLewis wants to merge 24 commits into
mainfrom
add_query_endpoint

Conversation

@MialLewis

@MialLewis MialLewis commented Jan 29, 2026

Copy link
Copy Markdown
Contributor

This PR adds an endpoint that allows trusted users to query the report database.

  • It requires a token which will only be shared selectively
  • It utilises a read only Postgres account
  • The database connection is read only

Closes #95

@MialLewis
MialLewis changed the base branch from main to make_changes_for_django_5110 January 29, 2026 17:00
Base automatically changed from make_changes_for_django_5110 to main January 30, 2026 13:34

@warunawickramasingha warunawickramasingha left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Proposed a few suggestions to improve the code, style and better handle exceptions + rate limiting rule at nginx level. Please check and apply as necessary.

Comment thread web/services/views.py Outdated
Comment thread web/services/views.py Outdated
Comment thread nginx/confs/reports_server.conf
Comment thread web/services/views.py Outdated
Comment thread web/services/views.py Outdated
Comment thread web/services/views.py Outdated
Comment thread nginx/confs/mantidreports.conf
Comment thread nginx/confs/reports_server.conf
Comment thread nginx/confs/reports_server.conf
Comment thread nginx/confs/mantidreports.conf Outdated
Renamed nginx server config file to be consistent
@github-actions github-actions Bot added the Has Conflicts Used by the bot to label pull requests that have conflicts label Jul 17, 2026
@github-actions

Copy link
Copy Markdown

👋 Hi, @MialLewis,

Conflicts have been detected against the base branch. Please rebase your branch against the base branch.

@github-actions github-actions Bot removed the Has Conflicts Used by the bot to label pull requests that have conflicts label Jul 17, 2026
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.

Provide an ingress to the underlying Postgres db for automated reports

2 participants