community
/
device-mgt-core
12
0
Fork 47
Code Issues Pull Requests 27 Packages Projects Releases Wiki Activity
Labels Milestones

Fix analytics publishing for Windows devices #240

Merged
pahansith merged 1 commits from navodzoysa/device-mgt-core:fix-windows-analytics into master 1 year ago
Conversation 0 Commits 1 Files Changed 2
navodzoysa commented 1 year ago
Collaborator

Purpose

Fixes https://roadmap.entgra.net/issues/10375

Changes

  • Check if the username taken from CarbonContext is not null and if it is null then get the device owner from device enrollment info and authorize if user is the device owner.

  • Handle Connection Refused exception by returning a 503 (Service Unavailable). This was added because if the reporting backend is down, the exception thrown will stop the device from syncing.

## Purpose Fixes https://roadmap.entgra.net/issues/10375 ## Changes * Check if the username taken from CarbonContext is not null and if it is null then get the device owner from device enrollment info and authorize if user is the device owner. * Handle Connection Refused exception by returning a 503 (Service Unavailable). This was added because if the reporting backend is down, the exception thrown will stop the device from syncing.
navodzoysa added 1 commit 1 year ago
pahansith merged commit 51f1bf1280 into master 1 year ago
The pull request has been merged as 51f1bf1280.
Sign in to join this conversation.
Reviewers
Request review
No reviewers
Labels
Apply labels
Clear labels
pending upstream
PR depends on pending upstream dependency
No Label pending upstream
Milestone
Set milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Assign users
Clear assignees
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: community/device-mgt-core#240
Write Preview
Loading…
Cancel
Save
There is no content yet.
Delete Branch 'navodzoysa/device-mgt-core:fix-windows-analytics'

Deleting a branch is permanent. It CANNOT be undone. Continue?

No
Yes