8000 fix "If-Equal" header predicate for new different from predicate for existing by thjaeckle · Pull Request #2116 · eclipse-ditto/ditto · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

fix "If-Equal" header predicate for new different from predicate for existing #2116

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
Merged
merged 1 commit into from
Feb 12, 2025
Merged

fix "If-Equal" header predicate for new different from predicate for existing #2116

merged 1 commit into from
Feb 12, 2025

Conversation

thjaeckle
Copy link
Member
  • causing that e.g. the existing Json did not contain "thingId" whereas the new one did

@thjaeckle thjaeckle added the bug label Feb 10, 2025
@thjaeckle thjaeckle added this to the 3.7.0 milestone Feb 10, 2025
@thjaeckle thjaeckle self-assigned this Feb 10, 2025
@thjaeckle thjaeckle force-pushed the bugfix/fix-if-equal-precondition-predicate branch from 28e2930 to bbe5741 Compare February 11, 2025 07:24
@thjaeckle
fix "If-Equal" header predicate for new different from predicate for …
ddb5dc2 
…existing

* causing that e.g. the existing Json did not contain "thingId" whereas the new one did
@thjaeckle thjaeckle force-pushed the bugfix/fix-if-equal-precondition-predicate branch from bbe5741 to ddb5dc2 Compare February 11, 2025 11:58
@thjaeckle thjaeckle modified the milestones: 3.7.0, 3.6.11 Feb 11, 2025
@thjaeckle
Copy link
Member Author

@alstanchev could you have a look?

alstanchev
alstanchev approved these changes Feb 12, 2025
View reviewed changes
Copy link
Contributor
@alstanchev alstanchev left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@thjaeckle thjaeckle merged commit 2de8ee8 into eclipse-ditto:master Feb 12, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@alstanchev alstanchev alstanchev approved these changes

Assignees

@thjaeckle thjaeckle

Labels
bug
Projects
Ditto Planning
Status: Done
Milestone
3.6.11
Development

Successfully merging this pull request may close these issues.
2 participants
@thjaeckle @alstanchev
0