After trying out different scenarios with the current version after latest update I found some nasty corner cases. For example IdentityServer can try to load all things based on scopes which caused confusion between collections. This PR adds collection prefix for document ids and keeps the uniqueness via the name. Now it's possible also to name, say identity resource and client, with the same name.
Decided to go with index queries instead of direct loads for now. Added more test coverage to fill missing bits.
DejanMilicic
commented
4 years ago
After trying out different scenarios with the current version after latest update I found some nasty corner cases. For example IdentityServer can try to load all things based on scopes which caused confusion between collections. This PR adds collection prefix for document ids and keeps the uniqueness via the name. Now it's possible also to name, say identity resource and client, with the same name.
Decided to go with index queries instead of direct loads for now. Added more test coverage to fill missing bits.