Skip to content

feat: additional info when started --without-auth#26338

Merged
praveen-influx merged 1 commit intomainfrom
praveen/without-auth-info
Apr 29, 2025
Merged

feat: additional info when started --without-auth#26338
praveen-influx merged 1 commit intomainfrom
praveen/without-auth-info

Conversation

@praveen-influx
Copy link
Copy Markdown
Contributor

  • return 405 message body to indicate the endpoints are disabled
  • extra log to say server has been started without auth

- return 405 message body to indicate the endpoints are disabled
- extra log to say server has been started without auth
@praveen-influx praveen-influx requested a review from a team April 29, 2025 00:11
Copy link
Copy Markdown
Contributor

@hiltontj hiltontj left a comment

Choose a reason for hiding this comment

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

The change looks good to me, but is there an issue open or slack thread for reference?

From what I can tell, this isn't changing any behaviour, just making existing behaviour clearer to the user in the logs / HTTP response.

@pauldix
Copy link
Copy Markdown
Member

pauldix commented Apr 29, 2025

Here: https://influxdata.slack.com/archives/C06BQA608J3/p1745885514983869

@praveen-influx
Copy link
Copy Markdown
Contributor Author

@hiltontj - This is the full thread from feedback channel https://influxdata.slack.com/archives/C084G9LR2HL/p1745880628511139. I should've opened an issue with relevant info!

@praveen-influx praveen-influx merged commit 08f2f85 into main Apr 29, 2025
12 checks passed
@hiltontj hiltontj deleted the praveen/without-auth-info branch April 29, 2025 13:14
hiltontj pushed a commit that referenced this pull request May 2, 2025
- return 405 message body to indicate the endpoints are disabled
- extra log to say server has been started without auth
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.

3 participants