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.

Sign up for GitHub

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

Jump to bottom

Zmiana flagi przechowywania klucza w Keychain #42

Closed
r3ggi opened this issue Apr 3, 2020 · 3 comments
Closed

Zmiana flagi przechowywania klucza w Keychain #42

r3ggi opened this issue Apr 3, 2020 · 3 comments
Assignees
@Axadiw @jerzyKleszcz

Comments

@r3ggi
Copy link

r3ggi commented Apr 3, 2020

Ze względów bezpieczeństwa proponuję zmianę flagi dostępności .always w pliku SecretsGeneratorSpecs na .afterFirstUnlockThisDeviceOnly.

Btw. Dostępność .always jest już oznaczana przez Apple jako deprecated - https://developer.apple.com/documentation/security/ksecattraccessiblealways
@jakublipinski
Copy link

@Axadiw @jerzyKleszcz zerkniecie na to?

@bdolewski
Copy link

Sprawdziłem cały kod na branch master w poszukiwaniu tego parametru, ponieważ nic nie mogłem znaleźć w pliku SecretsGeneratorSpecs czy KeychainProviderMock. Wygląda na to, że problem jest nieaktualny.

@jakublipinski
Copy link

Na prośbę @MateuszRomanow zamykam issues'y związane z wersją 1.0. Jeśli ten temat dotyczy również aktualnej wersji, proszę o otwarcie.

qLb pushed a commit that referenced this issue May 28, 2020
@mpodgorski
Merge pull request #42 from SafeSafe-app/feature/PSAFE-721
324eb46 
[PSAFE-721] Temporary Exposure Keys upload
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Axadiw Axadiw

@jerzyKleszcz jerzyKleszcz

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@jakublipinski @bdolewski @Axadiw @jerzyKleszcz @r3ggi