Skip to content

Conversation

@pitprok
Copy link

@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
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
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
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
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
Member

Choose a reason for hiding this comment

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

Copy link
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
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