simplesamlphp / SAML-tracer

Browser extension for examining SAML messages
https://addons.mozilla.org/nl/firefox/addon/saml-tracer/
BSD 2-Clause "Simplified" License
142 stars 39 forks source link

Name of the extension #39

Closed khlr closed 6 years ago

khlr commented 6 years ago

This may not be a big deal, but anyways ;-)

The other day I stumbled about the extension's name. It's different in various locations:

I'd suggest SAML-tracer. This term is already present in various places and wouldn't break existing hyperlinks. At least in the URLs I saw up to now, the hyphen was already present.

What do you think? Would it be a good idea to harmonize the name (e.g. for a better recognition value and unambiguity)? Or is this needless fuss?

jaimeperez commented 6 years ago

Hi @khlr!

I don't think this is a needless fuss in any way. Actually, I pretty much agree with you, as I find myself with the same trouble all over again. I don't have any special preferences, so whatever is easier to change /breaks less stuff works fine for me ๐Ÿ˜‰

khlr commented 6 years ago

Ok, then I'll do the necessary code changes in the near future. It's just a small thing ๐Ÿ˜‰

Is there a way to change the name at the Addons Page from SAML tracer to SAML-tracer, @jaimeperez? I mean that's not absolutely necessary, but would be the icing on the cake ๐Ÿ˜

image

Anyways, the URL will stay the same: https://addons.mozilla.org/firefox/addon/saml-tracer/

jaimeperez commented 6 years ago

Great, thanks!

Iโ€™m not entirely sure itโ€™s possible, but iโ€™ll check :-)

-- Jaime

On 26 Feb 2018, at 20:07, Jan Kรถhler notifications@github.com<mailto:notifications@github.com> wrote:

Ok, then I'll do the necessary code changes in the near future. It's just a small thing ๐Ÿ˜‰

Is there a way to change the name at the Addons Page from SAML tracer to SAML-tracer, @jaimeperezhttps://github.com/jaimeperez? I mean that's not absolutely necessary, but would be the icing on the cake ๐Ÿ˜

[image]https://user-images.githubusercontent.com/17160647/36689718-84edb732-1b30-11e8-88ba-b82109947cde.png

Anyways, the URL will stay the same: https://addons.mozilla.org/firefox/addon/saml-tracer/

โ€” You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/UNINETT/SAML-tracer/issues/39#issuecomment-368612771, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AB2kyBXkhdCvPOWAwh8vPWAw9hiLf1Gnks5tYwDCgaJpZM4SR1jO.

jaimeperez commented 6 years ago

... aaand there we have the icing ๐Ÿพ