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

keychain service #24

Closed
2 tasks done
jeonghomoon opened this issue Sep 18, 2022 · 0 comments · Fixed by #25
Closed
2 tasks done

keychain service #24

jeonghomoon opened this issue Sep 18, 2022 · 0 comments · Fixed by #25
Assignees
Labels
fix 버그 수정

Comments

@jeonghomoon
Copy link
Owner

jeonghomoon commented Sep 18, 2022

Description

Keychain Item Class의 이해 부족으로 #15 에서 kSecClassGenericPassword 외의 데이터를 저장하지 못하는 구조로 만들었었습니다.

kSecClassGenericPassword외의 데이터를 배재하여 현재 구조를 kSecClassGenericPassword 형태의 데이터를

마치 UserDefaults처럼 kSecAttrAccount-kSecValueData의 key-value 형태로 저장하도록 구조를 수정합니다.

요구사항

  1. Keychain initializer에 Item Class 선택부분을 제거하고, 테스트를 위해 service를 설정할 수 있어야함
  2. KeychainService의 CRUD 수정

TODO

  • Keychain initializer
  • Keychain CRUD

Environment

macOS: Monterey 12.5.1, Apple M1
iOS: 15.5, iPhone 13 mini
IDE: Xcode 13.4.1

Related: #15

@jeonghomoon jeonghomoon added the fix 버그 수정 label Sep 18, 2022
@jeonghomoon jeonghomoon self-assigned this Sep 18, 2022
jeonghomoon added a commit that referenced this issue Sep 18, 2022
Keychain
생성자에서 Keychain Item Class를 선택하는 구문을 제거하였습니다.
생성자에 service 값을 추가하여 테스트시 임시데이터 저장용으로 사용할 수 있도록 하였습니다.

KeychainService
Key 값을 account 값으로 사용하기 위해 Hashable이 아닌 String으로 고정하여 curd 메소드를 수정하였습니다.

Refs: #24
jeonghomoon added a commit that referenced this issue Sep 18, 2022
Keychain
생성자에서 Keychain Item Class를 선택하는 구문을 제거하였습니다.
생성자에 service 값을 추가하여 테스트시 임시데이터 저장용으로 사용할 수 있도록 하였습니다.

KeychainService
Key 값을 account 값으로 사용하기 위해 Hashable이 아닌 String으로 고정하여 curd 메소드를 수정하였습니다.

Refs: #24
This was referenced Sep 18, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix 버그 수정
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant