Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Missing informaiton for mutualTLS in v3.1.0 Spec #2475

Closed
philsturgeon opened this issue Feb 18, 2021 · 12 comments
Closed

Missing informaiton for mutualTLS in v3.1.0 Spec #2475

philsturgeon opened this issue Feb 18, 2021 · 12 comments
Labels
security: auth Authentication including overlap with authorization security

Comments

@philsturgeon
Copy link
Contributor

Just noticed there's no information about mutualTLS in the spec other than that it exists as a valid type.

@MikeRalphson
Copy link
Member

As intended. See lengthy discussions in #1004 - note specification extensions are allowed also. What fields do you think are missing?

@philsturgeon
Copy link
Contributor Author

I don’t know if it needs any fields but maybe an example or something. I’m just saying I have no idea how it works.

@MikeRalphson
Copy link
Member

It's just a hint to consoles / SDKs (and all other use-cases) that a client certificate must be obtained in some way and presented in the TLS connection.

@philsturgeon
Copy link
Contributor Author

Right, but unless we want to continue the years old OAI trend of making PR comments part of the spec/docs, could we consider making the specs have some information on how mutualTLS works? 😆

@MikeRalphson
Copy link
Member

unless we want to continue the years old OAI trend of making PR comments part of the spec/docs

Can you run that sentence by me one more time? I totally don't get it.

@philsturgeon
Copy link
Contributor Author

philsturgeon commented Feb 20, 2021 via email

@MikeRalphson
Copy link
Member

“the spec is clear but Darrel said this in an issue a few years ago” or “this issue says X and that pull request says Y but the spec doesn’t say anything so which is right”. I’d like to try and get away from that.

I see either @Relequestual's proposed use of Architectural Decision Records over at JSON Schema, or the publishing of some "technical notes" as a good way to capture additional information about how the sausage was made, without bloating the spec itself.

I'm still struggling to see what an example would add above the stated support for the type that's already in the spec. But, as ever, PRs for discussion are always welcomed!

@philsturgeon
Copy link
Contributor Author

philsturgeon commented Jun 21, 2021

I'm not looking for anything complex, just consistency. We show samples for other security schemes so lets add a sample for this one too. #2625

@cy6org
Copy link

cy6org commented Oct 15, 2021

I'm attempting to use the securityScheme type mutualTLS. Where/how to I provide the cert or cert chain I need in order to authenticate? The example (requested in this thread) should illustrate this detail.

If it's already documented and my weak google-foo can't find it, please advise.

@shriduttkothari
Copy link

I'm also attempting to use the securityScheme type mutualTLS.

But don't know how to use it, thee is not documentation available, and no sample or example avaiable.

Kindly help if anyone knows how to use securityScheme type mutualTLS!!

@handrews handrews added security security: auth Authentication including overlap with authorization labels Jan 28, 2024
@LasneF
Copy link
Member

LasneF commented Feb 28, 2024

given the subject is quite old ,
given a basic exemple has been provided in the merge request . Meaning you advertise that the API requires MutualTLS to be used .
given that for mutualTLS there must be direct and long communication between parties, to me the description field is good enough to details what is required

@lornajane
Copy link
Contributor

Thanks @LasneF ! Part of the problem here is that the example is in an unreleased spec version, but I think we do have the example requested, so I'll close the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
security: auth Authentication including overlap with authorization security
Projects
None yet
Development

No branches or pull requests

7 participants