Use `AND` instead of `OR` for multiple authentication options #1147

milosmns · 2019-12-25T19:27:57Z

Hi,

I see this was already reported in #577, but it seems like it's not fixed.

I have the following code:

"security": [
  {
    "AppID": []
  },
  {
    "AppSecret": []
  }
]

But it still generates:

So, I get OR there instead of AND. Even if this is intentional, I'd like to change to AND instead (at least with a configuration option).

This is my full HTML:

<!DOCTYPE html>
<html>
  <head>
    <title>My API</title>
    <!-- needed for adaptive design -->
    <meta charset="utf-8"/>
    <meta name="viewport" content="width=device-width, initial-scale=1">
    <link href="https://fonts.googleapis.com/css?family=Montserrat:300,400,700|Roboto:300,400,700" rel="stylesheet">

    <style>
      body {
        margin: 0;
        padding: 0;
      }
    </style>
  </head>
  <body>
    <redoc spec-url='my-api-on-gist.json' hide-download-button></redoc>
    <script src="https://cdn.jsdelivr.net/npm/redoc@next/bundles/redoc.standalone.js"></script>
  </body>
</html>

milosmns · 2019-12-25T22:18:41Z

I believe it's here:

redoc/src/components/SecurityRequirement/SecurityRequirement.tsx

Line 46 in 41a9b3c

content: ' ) OR ';

RomanHotsiy · 2020-01-10T14:00:03Z

This is per OAS spec, to get AND change your code to:

"security": [
  {
    "AppID": [],
    "AppSecret": []
  }
]

See here: https://github.com/OAI/OpenAPI-Specification/blob/master/versions/3.0.0.md#security-requirement-object

milosmns · 2020-01-10T14:16:15Z

@RomanHotsiy interesting, I'll take a look! Thanks!

RomanHotsiy closed this as completed Jan 10, 2020

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Use `AND` instead of `OR` for multiple authentication options #1147

Use `AND` instead of `OR` for multiple authentication options #1147

milosmns commented Dec 25, 2019

milosmns commented Dec 25, 2019

RomanHotsiy commented Jan 10, 2020

milosmns commented Jan 10, 2020

Use AND instead of OR for multiple authentication options #1147

Use AND instead of OR for multiple authentication options #1147

Comments

milosmns commented Dec 25, 2019

milosmns commented Dec 25, 2019

RomanHotsiy commented Jan 10, 2020

milosmns commented Jan 10, 2020

Use `AND` instead of `OR` for multiple authentication options #1147

Use `AND` instead of `OR` for multiple authentication options #1147