Adium

Opened 12 years ago

Closed 12 years ago

#8777 closed defect (cantreproduce)

Keychain handling: Endianess issues and entry names

Reported by: rschaeuble Owned by: nobody
Milestone: Adium 1.3 Component: Adium Core
Version: Severity: normal
Keywords: keychain passwords Cc:
Patch Status:

Description (last modified by Evan Schoenberg)

I have already reported this as ticket#6219. I was just trying to enter a comment there, showing that the ticket was wrongly closed, but submitting the comment always fails. So I'm reporting this in a new ticket now.

The second issue of ticket#6219 (Adium only using the default keychain) has been fixed. The other one, however, is still there:

~1. The names of the keychain entries still get increasing numbers (e.g. Adium.ICQ.1, Adium.ICQ.2 etc.). This requires me to enter accounts in the same order on all Macs that I'm using. I suggest to use some part of the account info (e.g. the name/ID) for keychain entry names (e.g. Adium.ICQ.my_user_name).~

  1. The URL of the keychain entries still has an endianess issue:

On my Intel Mac, the URLs look like MIdA:Adium.ICQ.1, on my PowerPC Mac they look like AdIM:Adium.ICQ.1 (the scheme part of the URL is different). That's a result of Adium's way of creating the URL, which is endianess-dependent. This could be easily fixed; however, already created keychain entries would have to be updated.

Change History (4)

comment:1 Changed 12 years ago by Jordan

Milestone: Adium X 1.3

comment:2 in reply to:  description Changed 12 years ago by Mike Hovan

I just wanted concur that having some way to identify which account a keychain entry is for would be useful. I wanted to delete a couple of my Adium account entries from the keychain (using KeychainAccess.app), but I could not figure out which account was which (Adium.AIM.4, Adium.AIM.5, Adium.AIM.7, etc... are not useful names for users). Showing the password within KeychainAcess.app helped for a couple of the accounts. But this seems a touch backwards. Do we even need any Adium identifying information in the keychain? It would be cool to be able to share credentials between multiple chat applications (like iChat).

comment:3 Changed 12 years ago by Evan Schoenberg

Description: modified (diff)
  1. The naming has been fixed as of Adium 1.3. The fix is not in 1.2.x.
  2. I do not see how to fix the endianness issue with the four-char code 'AdIM'. The time to fix this problem is now, as we'll be doing upgrade code on first launch of 1.3 anyways. Please submit a patch or sample code which demonstrates handling this properly.

comment:4 Changed 12 years ago by Jordan

Resolution: worksforme
Status: newclosed

The endianness issue with the four-char AdIM doesn't seem to be a problem anymore... perhaps it was fixed in Leopard at some point?

All of my entries on my Intel Mac show as AdIM not MIdA as is mentioned in the description.

Note: See TracTickets for help on using tickets.