Adium

Ticket #8702 (closed defect: fixed)

Opened 2 years ago

Last modified 11 months ago

Certificate alert text does not fit

Reported by: mrjoe Owned by: nobody
Milestone: Adium 1.2.2 Component: Adium Core
Version: 1.2b7 Severity: normal
Keywords: Cc:
Patch Status:

Description

One of my jabber account has a certificate which can not be validated. The alert which is displayed doesn't show the whole text. Seems to be a leopard issue as it looks ok on tiger.

Attachments

Afbeelding 1.png (65.3 KB) - added by mrjoe 2 years ago.
Screenshot showing the problem

Change History

Changed 2 years ago by mrjoe

Screenshot showing the problem

  Changed 2 years ago by jas8522

  • component changed from Adium UI to Localization
  • milestone set to Adium X 1.2

That language needs the translations to be reset and updated, what language are you using?

  Changed 2 years ago by evands

  • milestone changed from Adium X 1.2 to Waiting on Apple

Actually, that dialogue isn't under our control; it's entirely Apple's dialogue. I've seen that happen -once- and haven't been able to reproduce it; I don't see anything we can do about it.

follow-up: ↓ 6   Changed 2 years ago by evands

  • priority changed from normal to high
  • milestone changed from Waiting on Apple to Adium X 1.2.1

Rather, can do about it for 1.2. We should just implement our own dialogue and use the certificate view to display the pertinent information for 1.2.1.

  Changed 2 years ago by mrjoe

The language used is Dutch.

  Changed 2 years ago by masukomi

I can replicate this bug consistently. English language, but dvorak layout (which is treated as a language).

Adium v 1.2 (jan 5)

OS X 10.5.1

in reply to: ↑ 3 ; follow-up: ↓ 7   Changed 2 years ago by am

Replying to evands:

We should just implement our own dialogue and use the certificate view to display the pertinent information for 1.2.1.

I tried that before using Apple's dialog (due to the sheet issue). It was a real b*** with 200+ lines of code, didn't look as good as Apple's dialog, couldn't provide all the information their implementation displays, and you couldn't edit the trust settings (I have no idea how to re-implement that).

in reply to: ↑ 6   Changed 2 years ago by evands

Replying to am:

Replying to evands:

We should just implement our own dialogue and use the certificate view to display the pertinent information for 1.2.1.

I tried that before using Apple's dialog (due to the sheet issue). It was a real b*** with 200+ lines of code, didn't look as good as Apple's dialog, couldn't provide all the information their implementation displays, and you couldn't edit the trust settings (I have no idea how to re-implement that).

Hm. iChat definitely handles it this way, so it's in some way possible, though that says nothing as to how difficult it is :)

  Changed 2 years ago by am

Maybe they just took the (closed-source) code for SFCertificateTrustPanel, modified to their liking and compiled it into iChat.

  Changed 2 years ago by evands

Touché.

follow-up: ↓ 11   Changed 2 years ago by masukomi

I assume this is probably not a viable solution, but what would happen if you just stuck a few blank lines up above the start of the real text? Would the dialog not just expand downwards and the lines you wanted to be seen get pushed into the realm of the visible?

in reply to: ↑ 10   Changed 2 years ago by am

  • component changed from Localization to libpurple

Replying to masukomi:

I assume this is probably not a viable solution, but what would happen if you just stuck a few blank lines up above the start of the real text?

The problem is that whenever the sheet does open properly, you'd have a lot of blank lines up there and it'd look just as weird. The real problem is that the sheet doesn't slide all the way done for some very strange reason. I've never seen this behavior anywhere else on Mac OS X.

  Changed 2 years ago by am

done -> down too early in the morning for me ;)

  Changed 2 years ago by masukomi

"...whenever the sheet does open properly..."

Does this particular sheet ever open properly in Adium? If not, then why not use the proposed solution? Someday Apple will fix whatever it is that's causing this and all you'll have to do is remove a couple blank lines. In the interim people would have a slightly odd looking, but readable, dialog. The alternative seems to be some even uglier code hack that would involve hours of work and would, probably, also have to be removed once Apple fixes whatever's causing this.

  Changed 2 years ago by am

I have it open properly here pretty frequently.

  Changed 2 years ago by evands

I delibeately have one account not set to always trust just to keep seeing it. Out of perhaps 250 times seeing it, I've seen it not come all the way out twice.

  Changed 2 years ago by evands

  • milestone changed from Adium X 1.2.1 to Adium X 1.2.2

  Changed 2 years ago by spike411

I can reproduce this every time I start Adium (1.2.1b1 currently, Leopard 10.5.1).

I have one other Jabber account enabled and one ICQ account disabled.

  Changed 2 years ago by evands

  • status changed from new to closed
  • resolution set to fixed

(In [22480])

  • Fixed shifted text in the certificate trust panel. The bolded title was never intended to show more than one line of text... the textview itself was shifting up due to attempts to size it to fit in some situations, while the sheet was coming out fine. We now use the informational text - a private method, used by iChat as well to set non-bolded informational text - for the detailed information. This not only fixes #8702 but looks a ton better.
  • Use the independent window rather than the account sheet again. I think the problem was the 0-pixel height of the content view the fake window was using; I've seen 0-sized windows act strangely before. We now use a 1-pixel height, which looks just as good.
  • Set the security policy of the certificate panel to the SSL policy, since that's the Right One. It has a description in the docs as "Basic X509 plus host name verification per RFC 2818" versus "Basic X509-style certificate evaluation" of the Apple Basic we were using before, for whatever that's worth.

  Changed 2 years ago by evands

(In [22481]) Merged [22480]:

  • Fixed shifted text in the certificate trust panel. The bolded title was never intended to show more than one line of text... the textview itself was shifting up due to attempts to size it to fit in some situations, while the sheet was coming out fine. We now use the informational text - a private method, used by iChat as well to set non-bolded informational text - for the detailed information. This not only fixes #8702 but looks a ton better.
  • Use the independent window rather than the account sheet again. I think the problem was the 0-pixel height of the content view the fake window was using; I've seen 0-sized windows act strangely before. We now use a 1-pixel height, which looks just as good.
  • Set the security policy of the certificate panel to the SSL policy, since that's the Right One. It has a description in the docs as "Basic X509 plus host name verification per RFC 2818" versus "Basic X509-style certificate evaluation" of the Apple Basic we were using before, for whatever that's worth.

  Changed 11 months ago by zacw

  • component changed from libpurple to Adium Core

Removing 'libpurple' component.

Note: See TracTickets for help on using tickets.