Patch by chx.

Downloaded from pastebin via browser. Might fail to apply, and if it does, mayhaps the first patch to MAINTAINERS.txt that ever failed. No Omen involved. :P

CommentFileSizeAuthor
#3 do1167860-openid-maintainers.txt470 bytesheine
drupal.openid-wojtha.0.patch466 bytessun

Comments

Status: Needs review » Needs work

The last submitted patch, drupal.openid-wojtha.0.patch, failed testing.

wojtha’s picture

wojtha commented

subscribe 8-)

heine’s picture

heine commented
Status: Needs work » Needs review
StatusFileSize
new do1167860-openid-maintainers.txt470 bytes

I'm in favour.

sun’s picture

sun
Primary language German
Location Karlsruhe
commented

@wojtha would have to RTBC this patch.

wojtha’s picture

wojtha commented
Status: Needs review » Reviewed & tested by the community

Ok ... lets make the openid better ;-)

webchick’s picture

webchick
she/they
Primary language English
Location Vancouver 🇨🇦
commented
Version: 7.x-dev » 8.x-dev
Issue tags: +webchick approved

Let's get this in 8.x first (so we can sucker you into the next version too, HEE HEE). :) Welcome to MAINTAINERS.txt!

Background: wojtha is responsible for analyzing, debugging, and patching some major OpenID spec violations. He also sat on IRC with me for like 2 hours yesterday, so I could write up http://drupal.org/node/1120290#comment-4511796

dries’s picture

dries commented
Status: Reviewed & tested by the community » Fixed

That works for me -- wojtha has really stepped up to help with OpenID. Committed to 7.x and 8.x.

Thanks wojtha -- rock on!

Automatically closed -- issue fixed for 2 weeks with no activity.