Skip to content

Fixed Legal Disclaimer Response - #77

Open
DigiCertMike wants to merge 1 commit into
masterfrom
mrudloff-Updates
Open

DigiCertMike wants to merge 1 commit into
masterfrom
mrudloff-Updates

Conversation

@DigiCertMike

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI lite review requested due to automatic review settings October 9, 2026 17:16

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟡 Changes recommended

The current legal-notice default prevents certificate import; it must align with the documented execution contract.

1 open finding
What changed in this PR

Updates the FortiGate Linux AWR script’s legal-notice configuration and log destination.

Changes:

  • Sets LEGAL_NOTICE_ACCEPT to false.
  • Moves logs to /opt/digicert/logs.
File Summary
TrustLifeCycleManager/​Integrations/​Fortinet/​FortiGATE/​admin-webrequest-post-script/​Linux/​fortigate-awr.sh Updates legal-notice handling and logging configuration.

🧠 Review effort: Lite


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

# Configuration
LEGAL_NOTICE_ACCEPT="true" # Set to "true" to accept the legal notice and proceed with script execution
LOGFILE="/home/ubuntu/tlm_agent_3.1.16_linux64/awr-logs/fortigate.log"
LEGAL_NOTICE_ACCEPT="false" # Set to "true" to accept the legal notice and proceed with script execution
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.

2 participants