# Full name mapping when using SAML

**URL:** <https://meta.discourse.org/t/full-name-mapping-when-using-saml/100143>\
**Category:** SSO\
**Created:** [October 22, 2018, 7:14am UTC](https://meta.discourse.org/t/full-name-mapping-when-using-saml/100143 "2018-10-22T07:14:25Z")\
**Posts on this page:** 1\
**Showing post:** 3

<div class="post-metadata">

**Author:** ![david](https://sea3.discourse-cdn.com/meta/user_avatar/meta.discourse.org/david/32/157490_2.png) [@david](https://meta.discourse.org/u/david)\
**Post date:** [December 7, 2021, 11:26am UTC](https://meta.discourse.org/t/full-name-mapping-when-using-saml/100143/3 "2021-12-07T11:26:57Z")

</div>

Thanks for taking the time to dig into the code! That looks like a bug to me - we check the `fullName`, then replace it with `firstName` `lastName`. Line 133 in that snippet is totally useless 🤔

I’ve been working on the SAML plugin this week anyway, so here’s a fix:

[https://github.com/discourse/discourse-saml/pull/42](https://github.com/discourse/discourse-saml/pull/42)

Once that’s merged, the `fullName` will be correctly prioritised

---

_[View the full topic](https://meta.discourse.org/t/full-name-mapping-when-using-saml/100143)._