Skip to content

Add warning not to disable local auth#128399

Open
mfsmoot wants to merge 1 commit intoMicrosoftDocs:mainfrom
mfsmoot:mirabaismoot/add-local-auth-warning
Open

Add warning not to disable local auth#128399
mfsmoot wants to merge 1 commit intoMicrosoftDocs:mainfrom
mfsmoot:mirabaismoot/add-local-auth-warning

Conversation

@mfsmoot
Copy link
Copy Markdown

@mfsmoot mfsmoot commented Apr 10, 2026

No description provided.

@prmerger-automator
Copy link
Copy Markdown
Contributor

@mfsmoot : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change.

@learn-build-service-prod
Copy link
Copy Markdown
Contributor

Learn Build status updates of commit ef918b9:

✅ Validation status: passed

File Status Preview URL Details
articles/iot-hub-device-update/device-update-resources.md ✅Succeeded

For more details, please refer to the build report.

@ttorble ttorble requested a review from Copilot April 13, 2026 14:23
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adds documentation guidance warning users not to disable IoT Hub local authentication because Device Update depends on it.

Changes:

  • Adds an [!NOTE] callout warning against disabling local authentication on IoT Hub.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

In order for Device Update to receive change notifications from IoT Hub, Device Update integrates with the built-in Event Hubs. The IoT Hub will be configured automatically as part of the resource creation process with the required message routes, consumer groups, and access policy required to communicate with IoT devices.

> [!NOTE]
> Do not disable local authentication on the IoT Hub. Device Update requires local authentication to be enabled.
Copy link

Copilot AI Apr 13, 2026

Choose a reason for hiding this comment

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

The note explains the requirement but doesn’t direct readers to where the setting is or what will break if they disable it. Consider adding a brief pointer to the relevant IoT Hub setting (e.g., in Azure portal/Twin/CLI) and a short consequence statement (for example, Device Update service-to-hub integration or change-notification ingestion fails), so readers can act on the guidance.

Suggested change
> Do not disable local authentication on the IoT Hub. Device Update requires local authentication to be enabled.
> Do not disable **local authentication** on the linked IoT hub. In the Azure portal, check the IoT hub authentication settings and make sure local authentication remains enabled. Device Update requires local authentication for its service-to-hub integration; if you disable it, Device Update can fail to ingest IoT Hub change notifications and related Device Update operations may stop working.

Copilot uses AI. Check for mistakes.
@ttorble
Copy link
Copy Markdown
Contributor

ttorble commented Apr 13, 2026

@cwatson-cat

Can you review the proposed changes?

IMPORTANT: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator Bot added the aq-pr-triaged tracking label for the PR review team label Apr 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants