Skip to content

Conversation

@vsevel
Copy link
Contributor

@vsevel vsevel commented May 26, 2020

Follow up to #9552 and this specific #9552 (comment)
Rename credentials-provider-type into credentials-provider-name.
Discuss whether the existing name should be renamed into id.

/cc @sberyozkin @geoand

@geoand
Copy link
Contributor

geoand commented May 26, 2020

Seems reasonable to me ( obviously since I'm the one that originally brought it up 😆 )

@vsevel
Copy link
Contributor Author

vsevel commented May 26, 2020

Seems reasonable to me ( obviously since I'm the one that originally brought it up 😆 )

sure ;)

just to challenge before we close it, I think we should rename the credentials provider name to id or alias, so that we do not confuse it with the credentials provider bean name. I could see people getting very confused (if type did not speak to you, at least it was different from name).
what do you think @geoand ?

@geoand
Copy link
Contributor

geoand commented May 26, 2020

Yeah, that makes sense to me. @sberyozkin WDYT?

Copy link
Member

@gsmet gsmet left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Let's go with it. It's better than before.

@gsmet gsmet merged commit c169218 into quarkusio:master May 26, 2020
@gsmet
Copy link
Member

gsmet commented May 26, 2020

I will backport it.

@vsevel
Copy link
Contributor Author

vsevel commented May 26, 2020

Let's go with it. It's better than before.

the good news is that if we want to go ahead and rename the name field to id, this won't be a breaking change. let me know if you think it is worthwhile. otherwise I will keep it as is.

@vsevel vsevel deleted the credentials_rename_field branch May 26, 2020 19:26
@gsmet gsmet added this to the 1.5.0.Final milestone May 26, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants