Skip to content

Add alert for changed REST response codes in 5.2.0#1874

Open
Nadahar wants to merge 2 commits into
openhab:mainfrom
Nadahar:rest-response
Open

Add alert for changed REST response codes in 5.2.0#1874
Nadahar wants to merge 2 commits into
openhab:mainfrom
Nadahar:rest-response

Conversation

@Nadahar

@Nadahar Nadahar commented Feb 12, 2026

Copy link
Copy Markdown
Contributor

This is to notify users about the changes made in openhab/openhab-core#5236. I was waiting for a decision for openhab/openhab-core#5330 as well, but since nothing seems to happen there, I decided to get this done before I forget it.

Signed-off-by: Ravi Nadahar <nadahar@rediffmail.com>
@Nadahar Nadahar requested a review from a team as a code owner February 12, 2026 19:57
@Nadahar

Nadahar commented Feb 12, 2026

Copy link
Copy Markdown
Contributor Author

@florian-h05 Is this sufficient/appropriate?

@florian-h05 florian-h05 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.

I'd put this before the corona stats entry and prefix it with CORE:

ALERT;CORE: GET /actions/{thingUID} REST API endpoint has changed HTTP status code from 404 to 204 if no entries exist
ALERT;CORE: GET /channel-types/{channelTypeUID} and GET /thing-types/{thingTypeUID} REST API endpoints have changed HTTP status code from 204 to 404 if no result is found

@Nadahar

Nadahar commented Feb 13, 2026

Copy link
Copy Markdown
Contributor Author

I'd put this before the corona stats entry and prefix it with CORE:

I have never seen how these alerts are presented, so I don't know what they produce. Are there "codes" that trigger a certain "presentation", or is it simply printed as-is as text?

Updated alert messages for REST API endpoint status code changes.

Signed-off-by: Nadahar <Nadahar@users.noreply.github.com>
@florian-h05

Copy link
Copy Markdown
Contributor

They are presented as text while upgrading the package. The package scripts read them and print them. The first part (ALERT, WARNING) controls how they are presented (Color), the rest is just plain text.

@wborn wborn added the awaiting other PR Depends on another PR label Mar 7, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting other PR Depends on another PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants