cyberark / conjur-openapi-spec

OpenAPI v3 specification for Conjur / DAP v10+
https://conjur.org
Apache License 2.0
11 stars 2 forks source link

Fix Roles API test failures #225

Closed john-odonnell closed 1 year ago

john-odonnell commented 1 year ago

Desired Outcome

Fix the OpenAPI description CI build.

Implemented Changes

Connected Issue/Story

CNJR-1816

Definition of Done

At least 1 todo must be completed in the sections below for the PR to be merged.

Changelog

Test coverage

Documentation

Behavior

Security

john-odonnell commented 1 year ago

Tried and failed to fix the Kong example - we shouldn't let it block this PR. See my in-progress fix on the kong-fix branch.

Part of the issue is that the insomnia-inso npm package used to generate the Kong declarative config has been deprecated in favor of a new Inso CLI.

Once these are swapped, there's another issue when trying to make requests to Conjur through the Kong gateway:

<header>
  <h1>Blocked host: kong</h1>
</header>
<div id="container">
  <h2>To allow requests to kong make sure it is a valid hostname (containing only numbers, letters, dashes and dots), then add the following to your environment configuration:</h2>
  <pre>config.hosts &lt;&lt; "kong"</pre>
</div>

This is a result of Rails's Host Authorization middleware active in Conjur, restricting the set of accepted hostnames to only those listed here. Maybe we could open up the set of accepted hosts in Conjur's development config env.