Skip to content

Cut 5118 prd03 in#97

Merged
jworkmanjc merged 11 commits into
masterfrom
CUT-5118_prd03-in
May 21, 2026
Merged

Cut 5118 prd03 in#97
jworkmanjc merged 11 commits into
masterfrom
CUT-5118_prd03-in

Conversation

@jworkmanjc
Copy link
Copy Markdown
Contributor

@jworkmanjc jworkmanjc commented May 11, 2026

Issues

  • CUT-5118 - IN/ Prd03 Environment Support

What does this solve?

This release adds support for the prd03 region through the SDKs

Is there anything particularly tricky?

How should this be tested?

Screenshots


Note

Medium Risk
Adds a new regional host option that changes how SDKs select API/console endpoints, which could impact request routing if misconfigured. CI build behavior is also made conditional via a new label-controlled flag, slightly increasing workflow variability.

Overview
Adds IN/prd03 region support across the PowerShell SDKs by expanding host selection/parameterized-host enums (e.g., api.in/console.in) and updating user-facing environment guidance.

Updates the build pipeline to allow skipping ApiTransform.ps1 during CI when the PR has the skip-api-transform label (plumbed through to BuildAutoRest -RunApiTransform). Also bumps module/package versions for DirectoryInsights (0.2.0), V1 (0.2.0), and V2 (0.3.0) and updates release notes/docs accordingly.

Reviewed by Cursor Bugbot for commit cc0c5f8. Bugbot is set up for automated code reviews on this repo. Configure here.

@jworkmanjc jworkmanjc added V1 Release of the JumpCloud.SDK.V1 Module V2 Release of the JumpCloud.SDK.V2 Module DirectoryInsights Release of the JumpCloud.SDK.DirectoryInsights module minor Minor SDK Release Version skip-api-transform labels May 11, 2026
Comment thread SDKs/PowerShell/JumpCloud.SDK.DirectoryInsights/custom/ModuleIdentifier.cs Outdated
Comment thread SDKs/PowerShell/JumpCloud.SDK.V2/JumpCloud.SDK.V2.psd1 Outdated
Comment thread Custom/Module.cs
{
apiHostValue = "api.in";
consoleHostValue = "console.in";
}
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Substring .in match is overly broad for region detection

Medium Severity

The Contains(".in") check in the default branch and in the HostEnv translation logic can false-positive on values containing .internal, .info, .instance, etc. Unlike .eu or .stg01, .in is a very short and common substring. A user entering a custom value like console.internal.jumpcloud.com would be incorrectly routed to the India region. This affects all four copies of Module.cs.

Additional Locations (1)
Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit c3f6953. Configure here.

@jworkmanjc jworkmanjc requested a review from gweinjc May 11, 2026 17:30
gweinjc
gweinjc previously approved these changes May 11, 2026
Copy link
Copy Markdown
Contributor

@gweinjc gweinjc left a comment

Choose a reason for hiding this comment

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

Looks good to me, nice job

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

There are 2 total unresolved issues (including 1 from previous review).

Fix All in Cursor

Reviewed by Cursor Bugbot for commit cc0c5f8. Configure here.

Comment thread BuildAutoRest.ps1
@lucasmendes-jc lucasmendes-jc self-requested a review May 21, 2026 14:29
@jworkmanjc jworkmanjc merged commit 3d8201b into master May 21, 2026
18 checks passed
@jworkmanjc jworkmanjc deleted the CUT-5118_prd03-in branch May 21, 2026 16:05
@jworkmanjc jworkmanjc deployed to PublishToPSGallery May 21, 2026 16:06 — with GitHub Actions Active
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DirectoryInsights Release of the JumpCloud.SDK.DirectoryInsights module minor Minor SDK Release Version skip-api-transform V1 Release of the JumpCloud.SDK.V1 Module V2 Release of the JumpCloud.SDK.V2 Module

Development

Successfully merging this pull request may close these issues.

4 participants