Skip to content

Add optional "serviceName" property to all service methods - #82

Open
kiwi-josh wants to merge 1 commit into
martinkasa:masterfrom
kiwi-josh:feature/optional-service-name
Open

Add optional "serviceName" property to all service methods#82
kiwi-josh wants to merge 1 commit into
martinkasa:masterfrom
kiwi-josh:feature/optional-service-name

Conversation

@kiwi-josh

Copy link
Copy Markdown

Resolves: #51

Optionally allow serviceName to be provided when calling the methods, defaulting to the existing "cap_sec" if not provided to maintain backwards compatibility

… defaulting to the existing "cap_sec" if not provided to maintain backwards compatibility
@kiwi-josh
kiwi-josh force-pushed the feature/optional-service-name branch from 37d3632 to 3d2e6ca Compare September 24, 2023 21:19
@de-dan

de-dan commented May 8, 2024

Copy link
Copy Markdown

+1 but

  • Android implementation missing
  • I get that in the iOS implementation your new serviceName property is actually used as serviceName for the KeychainWrapper, but in Web the property is used for the storage entry key. So I would suggest a more global naming like prefix or keyPrefix

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.

Please add the ability to specify the serviceName

2 participants