Skip to content

Fix incongruent NAIP authentication scheme example#45

Closed
m-mohr with Copilot wants to merge 3 commits into
mainfrom
copilot/fix-naip-example-scheme
Closed

Fix incongruent NAIP authentication scheme example#45
m-mohr with Copilot wants to merge 3 commits into
mainfrom
copilot/fix-naip-example-scheme

Conversation

Copilot AI commented Jul 7, 2026

Copy link
Copy Markdown

The NAIP Item example had an authentication scheme named "none" with description "Free access without restrictions" but incorrectly specified type: "http" with scheme: "basic", which requires credentials. This created an internal inconsistency where the scheme name and description suggested no authentication while the type indicated basic auth.

Changes

  • Updated the "none" scheme to use custom type: "none" instead of type: "http"
  • Removed the contradictory scheme: "basic" field

Before

"none": {
  "type": "http",
  "scheme": "basic",
  "description": "Free access without restrictions"
}

After

"none": {
  "type": "none",
  "description": "Free access without restrictions"
}

The fix leverages the specification's support for custom authentication scheme types, ensuring the scheme name, type, and description are now congruent and accurately represent unrestricted access.

Copilot AI linked an issue Jul 7, 2026 that may be closed by this pull request
… NAIP example

Changed the authentication scheme from 'http' type with 'basic' scheme to a custom
'none' type, making the scheme name, type, and description congruent with the
intent of free access without restrictions. This resolves the incongruence where
the scheme was named 'none' and described as 'free access' but used basic
authentication which requires credentials."
Copilot AI changed the title [WIP] Fix incongruence in NAIP example scheme Fix incongruent NAIP authentication scheme example Jul 7, 2026
Copilot finished work on behalf of m-mohr July 7, 2026 15:48
Copilot AI requested a review from m-mohr July 7, 2026 15:48
@m-mohr

m-mohr commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

@copilot Don't use none, use basic and make the example make sense for this case.

Changed the authentication scheme from custom "none" type to standard http "basic"
authentication scheme. Updated the scheme name from "none" to "basic" and the
description from "Free access without restrictions" to "Requires Basic
Authentication" to make the example coherent and properly represent basic auth
requirements.

Copilot AI commented Jul 7, 2026

Copy link
Copy Markdown
Author

@copilot Don't use none, use basic and make the example make sense for this case.

Done! Changed to use basic authentication with a coherent description. The scheme is now named "basic", uses type: "http" with scheme: "basic", and has the description "Requires Basic Authentication" to accurately reflect the authentication requirement. Commit: e59b8ed

Copilot finished work on behalf of m-mohr July 7, 2026 15:54
@m-mohr m-mohr marked this pull request as ready for review July 7, 2026 16:01
@m-mohr

m-mohr commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

@co

@m-mohr m-mohr closed this Jul 7, 2026
Copilot stopped work on behalf of m-mohr due to an error July 7, 2026 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

NAIP example with scheme of "none" is incongruent

2 participants