Skip to content

Proper url spec for ocm server url - #248

Merged
MahdiBaghbani merged 4 commits into
developfrom
mahdi/directory-service-ocm-addr
Aug 20, 2025
Merged

Proper url spec for ocm server url#248
MahdiBaghbani merged 4 commits into
developfrom
mahdi/directory-service-ocm-addr

Conversation

@MahdiBaghbani

@MahdiBaghbani MahdiBaghbani commented Aug 19, 2025

Copy link
Copy Markdown
Member

I'm intentionally not updating the Discovery endpoint (and any related fields) in this PR. This needs a brief design discussion first. For reference:

* REQUIRED: endPoint (string) - The URI of the OCM API available at
this endpoint.  Example: `"https://my-cloud-storage.org/ocm"`

Once we reach consensus, I'll submit a follow up patch with the corresponding edits.

This PR is a result of discussion in #244

@mickenordin mickenordin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Great 👍

@glpatcern glpatcern left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good! Just a couple of changes

Comment thread IETF-RFC.md Outdated
Comment thread IETF-RFC.md Outdated
MahdiBaghbani and others added 3 commits August 20, 2025 10:27
Co-authored-by: Giuseppe Lo Presti <giuseppe.lopresti@cern.ch>
Co-authored-by: Giuseppe Lo Presti <giuseppe.lopresti@cern.ch>
@KrausMatthias

Copy link
Copy Markdown
Contributor

Maybe we should add something in Discovery on how to handle https vs http. Default to https only, rejecting any federated urls with http or OCM Servers only discoverable via http. Allow an Opt-In for http downgrades/fallback?

One might also leave that up to implementations, just something I've been thinking about.

@glpatcern
glpatcern self-requested a review August 20, 2025 07:49
@glpatcern

Copy link
Copy Markdown
Member

Maybe we should add something in Discovery on how to handle https vs http. Default to https only, rejecting any federated urls

Yes, that's what Mahdi put in the subject of this PR, leaving Discovery as a separate point.

My take here would be to provide a recommendation (the obvious one...) and leave it to the implementations. De facto the only room for http is for testing purposes, and even there we have certificates and https in the OCM Test Suite, Paired with the Security considerations where we recommend httpsig, it does not leave much choice,

@MahdiBaghbani
MahdiBaghbani merged commit d185cce into develop Aug 20, 2025
2 checks passed
@MahdiBaghbani
MahdiBaghbani deleted the mahdi/directory-service-ocm-addr branch August 20, 2025 09:21
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.

4 participants