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.

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

Implement retrieveStorageIdentifier for MongoTokenStore #49

Closed
smcvb opened this issue Jan 14, 2021 · 0 comments · Fixed by #139
Closed

Implement retrieveStorageIdentifier for MongoTokenStore #49

smcvb opened this issue Jan 14, 2021 · 0 comments · Fixed by #139

Comments

@smcvb
Copy link
Member

smcvb commented Jan 14, 2021

Feature Description

The MongoTokenStore should implement the TokenStore#retrieveStorageIdentifier method.
This will ensure that an actual identifier is returned instead of the default --unknown-- (as is dictated by the TrackingEventProcessor).

Current Behaviour

The TokenStore identifier is noted as --unknown--.

Wanted Behaviour

The TokenStore identifier should return an actual identifier.

@smcvb smcvb added this to the Release 4.5 milestone Jan 14, 2021
@smcvb smcvb self-assigned this Oct 19, 2021
smcvb added a commit that referenced this issue Oct 19, 2021
Implement the retrieveStorageIdentifier() method. Do so by first
checking if a ConfigToken is present within the trackingTokensCollection
If it isn't, introduce a new token into the collection. In either case,
return the existing or created ConfigToken to return the ID from it as
the result of retrieveStorageIdentifier

#49
smcvb added a commit that referenced this issue Oct 19, 2021
Whilst touching the MongoTokenStoreTest, a secure XStreamSerializer
should be added. Use this in the token store test, as well as in other
test cases.

#49
smcvb added a commit that referenced this issue Oct 25, 2021
[#49] Implement `retrieveStorageIdentifier` for MongoTokenStore
smcvb added a commit that referenced this issue Oct 25, 2021
Enforce configuration of an XStream instance

#49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant