Make WordPress Core

Opened 17 years ago

Closed 17 years ago

#3917 closed defect (bug) (fixed)

Change wp-login.php to allow web single sign on tools to work

Reported by: anhill's profile anhill Owned by:
Milestone: 2.2 Priority: low
Severity: normal Version: 2.1.2
Component: Administration Keywords: has-patch
Focuses: Cc:

Description

Due to a limitation in JavaScript, web single sign-on tools (in this case Novell Access Manager) are unable to do a form-fill SSO authentication to WordPress. The reason is that several of the form elements are named and ID'd "submit". JavaScript gets confused, and won't execute the form fill code, which includes a "document.forms[0].submit()" function to enable the transparent login.

This patch is very simple - it changes name="submit" to name="wp-submit" and id="submit" to id="wp-submit" in wp-login.php.

Hopefully I made the problem clear - feel free to let me know if there are any questions!

Attachments (1)

submit.diff (1.6 KB) - added by anhill 17 years ago.
Patch to wp-login.php changing submit attributes

Download all attachments as: .zip

Change History (8)

@anhill
17 years ago

Patch to wp-login.php changing submit attributes

#1 @anhill
17 years ago

  • Milestone changed from 2.3 to 2.2

#2 @westi
17 years ago

  • Component changed from Security to Administration
  • Keywords has-patch added
  • Summary changed from wp-login.php change to allow web SSO tools to work to Change wp-login.php to allow web single sign on tools to work

Patch looks good.

Should we give them all unique names instead though?

#3 @anhill
17 years ago

I think the ones named submit are the only ones that would ever cause a problem - the rest should be fine.

#4 @foolswisdom
17 years ago

anhill, I think westi is asking if each of those three entries you include in your patch should be given unique names?

#5 @JeremyVisser
17 years ago

No, because they're the same buttons in separate forms.

#6 @rob1n
17 years ago

I'm with Jeremy on this one.

#7 @ryan
17 years ago

  • Resolution set to fixed
  • Status changed from new to closed

(In [4981]) Change wp-login submit names and ids. Props anhill. fixes #3917

Note: See TracTickets for help on using tickets.