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

The "_policy" is persisted as part of "MergeThing" event when initial inline policy was provisioned #1691

Closed
thjaeckle opened this issue Jul 13, 2023 · 0 comments · Fixed by #1695
Closed

The "_policy" is persisted as part of "MergeThing" event when initial inline policy was provisioned #1691

thjaeckle opened this issue Jul 13, 2023 · 0 comments · Fixed by #1695
Assignees
@thjaeckle
Labels
bug
Milestone
3.3.5

Comments

@thjaeckle
Copy link
Member

Since 3.3.0 it is possible to create a thing with a MergeThing (PATCH Thing) command - and it also is possible to provide an inline policy to be created for the "create thing" case.

This "_policy" field however seems to be persisted in the Things event journal - which is definitely not wanted.
Also causes not much of damage, only causes more MongoDB storage.

Should be fixed anyways ..
@thjaeckle thjaeckle added the bug label Jul 13, 2023
@thjaeckle thjaeckle added this to the 3.3.5 milestone Jul 13, 2023
thjaeckle added a commit that referenced this issue Jul 17, 2023
@thjaeckle
#1691 fixed persisting inline "_policy" in ThingMerged events
df4462c 
Signed-off-by: Thomas Jäckle <thomas.jaeckle@beyonnex.io>
@thjaeckle thjaeckle mentioned this issue Jul 17, 2023
Merged
@thjaeckle thjaeckle self-assigned this Jul 18, 2023
thjaeckle added a commit that referenced this issue Jul 21, 2023
@thjaeckle
Merge pull request #1695 from eclipse-ditto/bugfix/remove-inline-poli…
d2f8d36 
…cy-from-persisted-thing-merged-event

#1691 fixed persisting inline "_policy" in ThingMerged events
thjaeckle added a commit that referenced this issue Jul 21, 2023
@thjaeckle
#1691 fixed persisting inline "_policy" in ThingMerged events
c593cde 
Signed-off-by: Thomas Jäckle <thomas.jaeckle@beyonnex.io>
(cherry picked from commit df4462c)
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@thjaeckle thjaeckle

Labels
bug
Projects
Ditto Planning
Status: Done
Milestone
3.3.5
Development

Successfully merging a pull request may close this issue.

#1691 fixed persisting inline "_policy" in ThingMerged events eclipse-ditto/ditto
1 participant
@thjaeckle