Skip to content

Add status code 423 Locked#1321

Open
pitprok wants to merge 2 commits intojakartaee:mainfrom
pitprok:add-code-423-locked
Open

Add status code 423 Locked#1321
pitprok wants to merge 2 commits intojakartaee:mainfrom
pitprok:add-code-423-locked

Conversation

@pitprok
Copy link
Copy Markdown

@pitprok pitprok commented Jun 30, 2025

This adds the Locked status code for cases where the requested REST resource has been locked.
For example, if a resource is being updated and we need to lock it while the update is in progress.

@pitprok
Copy link
Copy Markdown
Author

pitprok commented Jun 30, 2025

I have created an Eclipse Foundation account, linked my Github account and signed the Eclipse Contributor agreement but the check is still failing, I don't know what's missing.

@jamezp
Copy link
Copy Markdown
Member

jamezp commented Jun 30, 2025

It doesn't look like your commit is signed https://api.eclipse.org/git/eca/status/gh/jakartaee/rest/1321

* 423 Locked, see <a href="https://datatracker.ietf.org/doc/html/rfc4918#section-9.8.5">RFC 4918: HTTP Extensions
* for Web Distributed Authoring and Versioning</a>.
*
* @since 3.2
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This should be @since 5.0

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

done

*/
EXPECTATION_FAILED(417, "Expectation Failed"),
/**
* 423 Locked, see <a href="https://datatracker.ietf.org/doc/html/rfc4918#section-9.8.5">RFC 4918: HTTP Extensions
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

done

@pitprok
Copy link
Copy Markdown
Author

pitprok commented Jul 1, 2025

It doesn't look like your commit is signed https://api.eclipse.org/git/eca/status/gh/jakartaee/rest/1321

It looks like it needed some time in order to detect the change, now it says "The author(s) of the pull request is covered by necessary legal agreements in order to proceed!"

@pitprok pitprok requested a review from jamezp July 2, 2025 06:06
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.

2 participants