Skip to content

fix(ocm-add-share): adjust domain validation#61310

Open
cristianscheid wants to merge 1 commit into
masterfrom
fix/noid/ocm-add-share-validation
Open

fix(ocm-add-share): adjust domain validation#61310
cristianscheid wants to merge 1 commit into
masterfrom
fix/noid/ocm-add-share-validation

Conversation

@cristianscheid

Copy link
Copy Markdown
Member
  • Resolves: #

Summary

  • adds validation for domains from owner or sharedBy fields and checks if they contain any suspicious characters, like letters which are almost identical visually but are Unicode characters from different sets (e.g. "micrоsoft.com" spelled with Cyrillic "о" character)

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

@cristianscheid cristianscheid self-assigned this Jun 15, 2026
@cristianscheid cristianscheid requested a review from a team as a code owner June 15, 2026 16:32
@cristianscheid cristianscheid removed the request for review from a team June 15, 2026 16:32
@cristianscheid cristianscheid added the 3. to review Waiting for reviews label Jun 15, 2026
@cristianscheid cristianscheid requested a review from sorbaugh June 15, 2026 16:32
Signed-off-by: Cristian Scheid <cristianscheid@gmail.com>
@cristianscheid cristianscheid force-pushed the fix/noid/ocm-add-share-validation branch from 93b00df to 196a15b Compare June 15, 2026 16:35
@cristianscheid cristianscheid requested a review from come-nc June 15, 2026 17:22
@cristianscheid cristianscheid changed the title Fix/noid/ocm add share validation fix(ocm-add-share): enhance validation Jun 15, 2026
@cristianscheid cristianscheid changed the title fix(ocm-add-share): enhance validation fix(ocm-add-share): adjust domain validation Jun 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant