-
Notifications
You must be signed in to change notification settings - Fork 1k
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.
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
[MozillaSecurityBridge] Make the URI unique by adding timestamp #1005
Conversation
Mandatory for some feed reader (Kriss Feed for instance)
Thanks for this PR! This PR is a workaround for the issues discussed in #977 but only works for one bridge. The proposed fix is to introduce a new (optional) parameter to I'll get back to you once it is implemented. Leave this PR open in the mean time. |
if UID is mandatory for RSS-Bridge, the unicity of the URI is also mandatory for some reader (like kriss feed).
Hi @logmanoriginal just for me to know, do I need to do anything special for this PR to be approved ? |
Thanks @teromene for stepping in. Sorry @Nono-m0le I've been ill the past weeks, so I couldn't merge. |
No problem at all, important is the community can now hopefully enjoy it :) |
'uid' represents the unique id for a feed item. This item is null by default and can be set to any string value. The provided string value is always hashed to sha1 to make it the same length in all cases. References RSS-Bridge#977, RSS-Bridge#1005
…Bridge#1005) * added unique UID + URI if UID is mandatory for RSS-Bridge, the unicity of the URI is also mandatory for some reader (like kriss feed).
Mandatory for some feed reader (Kriss Feed for instance)