# SSO add\_groups should be case-insensitive

**URL:** https://meta.discourse.org/t/sso-add-groups-should-be-case-insensitive/67183
**Category:** Bug
**Created:** [July 31, 2017, 5:48pm UTC](https://meta.discourse.org/t/sso-add-groups-should-be-case-insensitive/67183 "2017-07-31T17:48:22Z")
**Posts on this page:** 4
**Page:** 1

<div class="post-metadata">

### Author: ![pfaffman](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/pfaffman/32/120154_2.png) [@pfaffman](https://meta.discourse.org/u/pfaffman)
#### Post date: [July 31, 2017, 5:48pm UTC](https://meta.discourse.org/t/sso-add-groups-should-be-case-insensitive/67183/1 "2017-07-31T17:48:22Z")

</div>

I think I’ve finally figured out how to get Paid Memberships Pro to properly update group memberships with an SSO payload. My last problem was because I was careless about case of group names. And since I got the case wrong only once, it took me forever to find it.

Perhaps I should call this a feature request, but it cost me another 15 minutes to figure this one out, so I’m calling it a 🐛. 🙂

[https://github.com/discourse/discourse/blob/master/app/models/discourse\_single\_sign\_on.rb#L113](https://github.com/discourse/discourse/blob/master/app/models/discourse_single_sign_on.rb#L113)

---

<div class="post-metadata">

### Author: ![codinghorror](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/codinghorror/32/110067_2.png) [@codinghorror](https://meta.discourse.org/u/codinghorror)
#### Post date: [July 31, 2017, 11:48pm UTC](https://meta.discourse.org/t/sso-add-groups-should-be-case-insensitive/67183/2 "2017-07-31T23:48:49Z")

</div>

Sure @neil if this looks good, maybe add it? Case sensitivity is the 👹

---

<div class="post-metadata">

### Author: ![neil](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/neil/32/102150_2.png) [@neil](https://meta.discourse.org/u/neil)
#### Post date: [August 2, 2017, 4:57pm UTC](https://meta.discourse.org/t/sso-add-groups-should-be-case-insensitive/67183/3 "2017-08-02T16:57:13Z")

</div>

Fixed! Group lookup is case insensitive there now.

---

<div class="post-metadata">

### Author: ![codinghorror](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/codinghorror/32/110067_2.png) [@codinghorror](https://meta.discourse.org/u/codinghorror)
#### Post date: [August 2, 2017, 9:58pm UTC](https://meta.discourse.org/t/sso-add-groups-should-be-case-insensitive/67183/4 "2017-08-02T21:58:27Z")

</div>


