Skip to content

storage/http: expose ignore_uids config parameter#1135

Closed
pacien wants to merge 1 commit intopimutils:mainfrom
pacien:http-ignore-uids-param
Closed

storage/http: expose ignore_uids config parameter#1135
pacien wants to merge 1 commit intopimutils:mainfrom
pacien:http-ignore-uids-param

Conversation

@pacien
Copy link
Contributor

@pacien pacien commented Sep 1, 2024

This allows users to choose whether or not to ignore the UIDs coming
from WebCAL providers and instead use the item's content hash. This
remains the default behaviour.

In my case, my provider was generating stable UIDs but was patching the
export date in other fields, resulting in the hash of all events
changing at each synchronisation, having the opposite effect.

@pacien pacien force-pushed the http-ignore-uids-param branch 2 times, most recently from e6d7d54 to 66cd4a9 Compare September 1, 2024 10:39
This allows users to choose whether or not to ignore the UIDs coming
from WebCAL providers and instead use the item's content hash. This
remains the default behaviour.

In my case, my provider was generating stable UIDs but was patching the
export date in other fields, resulting in the hash of all events
changing at each synchronisation, having the opposite effect.
@pacien pacien force-pushed the http-ignore-uids-param branch from 66cd4a9 to 8f63da6 Compare December 8, 2024 18:30
@pacien
Copy link
Contributor Author

pacien commented Dec 8, 2024

Rebase and fixed merge conflict.

@WhyNotHugo
Copy link
Member

#1136 doesn't address the issue which you're describing?

@pacien
Copy link
Contributor Author

pacien commented Dec 9, 2024

Yes, #1136 covers that problem and more. I'll close this PR. Thanks!

@pacien pacien closed this Dec 9, 2024
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