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

allow copy keychain password data #601

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

profiles
Copy link

No description provided.

@NSExceptional
Copy link
Collaborator

You know debug description doesn't necessarily contain the full data right?

@profiles
Copy link
Author

You know debug description doesn't necessarily contain the full data right?

I don't know, but it's simple and useful for nsdata

@NSExceptional
Copy link
Collaborator

Is this really useful to you?

image

@profiles
Copy link
Author

Is this really useful to you?

image

this is description, not debugDescription

@profiles
Copy link
Author

Is this really useful to you?

image

switch tab to NSObject, and filter "debugDescription", you can see it

@NSExceptional
Copy link
Collaborator

Ah. I'm still not sure coping the plain string representation is the best option. What if you could airdrop the data as a file or copy a base64 encoded string?

@profiles
Copy link
Author

Ah. I'm still not sure coping the plain string representation is the best option. What if you could airdrop the data as a file or copy a base64 encoded string?

it's not the best option, but it's option which can fix a problem when the keychain NSData forced convert to NSString.
base64 string can reduce 2/3 bytes, but hex string is more explicitly.
as for airdrop, it's complex :(

@NSExceptional
Copy link
Collaborator

Do you mind me asking what you do with the data once you copy the hex string?

@profiles
Copy link
Author

Do you mind me asking what you do with the data once you copy the hex string?

view, check, export or analysis keychain data

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.

3 participants