Fix a regression in confirming trust when configuring repositories.

Review Request #12829 — Created Feb. 7, 2023 and submitted

Information

Review Board
release-4.0.x

Reviewers

Review Board 4 introduced a regression when confirming trust in
repository certificates or SSH keys. While this isn't used particularly
often (as most people now interface with services like GitHub or with
repositories that don't require this kind of verification), it has been
noticed recently by a couple different parties.

The problem was that, when clicking "I trust this host," blank form data
was being sent along. This happened because the fields on the form are
disabled when this prompt is shown, and disabled fields aren't sent in
form data.

We had code meant to deal with this. It re-enabled the disabled fields
when confirming. However, a restyle removed the old class name on the
alert, preventing this from working.

This change simply adds back the old class name, allowing this to work
again. We're not worrying about modern naming at this stage, since the
whole form is being rewritten for Review Board 6.

Triggered a host verification prompt. Verified that the form fields
were no longer disabled and the form data was properly sent along.

Summary ID
Fix a regression in confirming trust when configuring repositories.
Review Board 4 introduced a regression when confirming trust in repository certificates or SSH keys. While this isn't used particularly often (as most people now interface with services like GitHub or with repositories that don't require this kind of verification), it has been noticed recently by a couple different parties. The problem was that, when clicking "I trust this host," blank form data was being sent along. This happened because the fields on the form are disabled when this prompt is shown, and disabled fields aren't sent in form data. We had code meant to deal with this. It re-enabled the disabled fields when confirming. However, a restyle removed the old class name on the alert, preventing this from working. This change simply adds back the old class name, allowing this to work again. We're not worrying about modern naming at this stage, since the whole form is being rewritten for Review Board 6.
c5f675e5d79229183037652885f87853c1951446
david
  1. Ship It!
  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-5.0.x (46a0773)
Loading...