Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fedora Adapter Optimistic Locking #454

Closed
tpendragon opened this issue Jul 18, 2018 · 2 comments
Closed

Fedora Adapter Optimistic Locking #454

tpendragon opened this issue Jul 18, 2018 · 2 comments
Assignees
@hackartisan

Comments

@tpendragon
Copy link
Collaborator

Depends on #453
Child of #286
@tpendragon tpendragon mentioned this issue Jul 18, 2018
Closed
@escowles
Copy link
Contributor

Fedora requests with the If-Unmodified-Since header (see [RFC2616)[https://www.w3.org/Protocols/rfc2616/rfc2616-sec14.html#sec14.28)) will fail if the resource has been modified after that date.

@escowles
Copy link
Contributor

The LDP gem has some related functionality, but it does not look like there is an easy way to pass the modified date. See samvera/ldp#99

@awead awead added in progress and removed ready labels Aug 3, 2018
@hackartisan hackartisan assigned hackartisan and unassigned awead and tpendragon Aug 6, 2018
hackartisan added a commit that referenced this issue Aug 6, 2018
@awead @tpendragon
@hackartisan @carolyncole
Add optimistic locking to Fedora
629fb6e 
fixes #454

Co-authored-by: Trey Pendragon <[email protected]>
Co-authored-by: Anna Headley <[email protected]>
Co-authored-by: Carolyn Cole <[email protected]>
hackartisan added a commit that referenced this issue Aug 6, 2018
@awead @tpendragon
@hackartisan @carolyncole
Add optimistic locking to Fedora
2c11226 
fixes #454

Co-authored-by: Trey Pendragon <[email protected]>
Co-authored-by: Anna Headley <[email protected]>
Co-authored-by: Carolyn Cole <[email protected]>
@awead awead mentioned this issue Aug 6, 2018
Merged
@escowles escowles mentioned this issue Aug 6, 2018
Closed
@escowles escowles removed the Review label Aug 6, 2018
jrgriffiniii pushed a commit that referenced this issue Aug 7, 2018
@awead @tpendragon
@hackartisan @carolyncole @jrgriffiniii
Add optimistic locking to Fedora
5844e51 
fixes #454

Co-authored-by: Trey Pendragon <[email protected]>
Co-authored-by: Anna Headley <[email protected]>
Co-authored-by: Carolyn Cole <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@hackartisan hackartisan

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@awead @hackartisan @escowles @tpendragon