Skip to content

Protection check to avoid overwriting of the audit file#9

Open
SiddhJog wants to merge 1 commit intomainfrom
fix/prevent-aidlc-overwrite-audit-file
Open

Protection check to avoid overwriting of the audit file#9
SiddhJog wants to merge 1 commit intomainfrom
fix/prevent-aidlc-overwrite-audit-file

Conversation

@SiddhJog
Copy link
Contributor

Issue 8: #8

added a protection check before the audit file is updated at the beginning of the workflow

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Copy link
Contributor

@raj-jain-aws raj-jain-aws left a comment

Choose a reason for hiding this comment

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

@SiddhJog Thanks for this fix.

  • Can you add an example of what happens before this fix?

  • One rule statement says ""...STOP and present protection message". How does the protection message look like? Do we provide guidance to user how to handle this situation?

@raj-jain-aws raj-jain-aws self-assigned this Dec 11, 2025
@wmatosjr
Copy link

These instructions are too verbose and can be simplified for higher likelihood of compliance. Let's discuss.

@raj-jain-aws
Copy link
Contributor

@wmatosjr Agreed to the verbosity concern. I think that's a bigger concern we need to handle in general, shed the fat first and then ensure that whatever new behaviors we put on later are as lean an possible.

raj-jain-aws pushed a commit that referenced this pull request Jan 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants