Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CoE Starter Kit - BUG] New way of gathering Audit Logs hard-coded to Commercial #6075

Closed
1 task done
osbornecc opened this issue Jul 14, 2023 · 6 comments
Closed
1 task done
Assignees
Labels
bug Something isn't working coe-starter-kit CoE Starter Kit issues

Comments

@osbornecc
Copy link

Does this bug already exist in our backlog?

  • I have checked and confirm this is a new bug.

Describe the issue

We noticed none of our audit log data was getting pulled using the new method. We had updated the environment variable "Audit Logs - Audience" to https://manage-gcc.office.com per instructions.

However, running the new Admin | Audit Logs | Sync Audit Logs (V2) flow did nothing. Looking into that flow, the HTTP action is hard coded to Commercial's audit log URL (this is just one example, I didn't look yet at all the others)

image

We can fix this ourselves, but wanted to let you know.
Thanks!

Expected Behavior

No response

What solution are you experiencing the issue with?

Core

What solution version are you using?

4.11

What app or flow are you having the issue with?

Admin | Audit Logs | Sync Audit Logs (V2)

What method are you using to get inventory and telemetry?

None

Steps To Reproduce

No response

Anything else?

No response

@osbornecc osbornecc added bug Something isn't working coe-starter-kit CoE Starter Kit issues labels Jul 14, 2023
@osbornecc
Copy link
Author

I also just looked at Admin | Audit Logs | Office 365 Management API Subscription. The HTTP actions there are also hard coded to Commercial. We already had a subscription from the prior way, so it didn't impact us.

@Jenefer-Monroe
Copy link
Collaborator

thanks for posting. looks like we missed it in this spot in all actions but caught the rest.

Fix is to put the Audit Logs - Audiance env var before the slash as shown here
image

And then turn the flow off and back on to get the env var updated.
Feel free to make the change locally and then you can just remove the unmanaged layer when you upgrade.

@Jenefer-Monroe
Copy link
Collaborator

local fixed for August

@osbornecc
Copy link
Author

@Jenefer-Monroe
Thanks! That's exactly what we did ;-)

@Jenefer-Monroe
Copy link
Collaborator

Go team! :)

@CoEStarterKitBot
Copy link
Collaborator

@osbornecc This has been fixed in the latest release. Please install the latest version of the toolkit following the instructions for installing updates. Note that if you do not remove the unmanaged layers as described there you will not receive updates from us.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working coe-starter-kit CoE Starter Kit issues
Projects
Archived in project
Development

No branches or pull requests

3 participants