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

[standardization] Copilot Logins, Redact PII, IP Geolocation #186

Merged
merged 12 commits into from
Mar 28, 2025

Conversation

mariocuomo
Copy link
Contributor

No description provided.

@KwachSean KwachSean requested a review from Copilot March 20, 2025 11:38
Copy link
Contributor

@Copilot 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

This PR standardizes and updates our Security Copilot plugins by introducing a new manifest for the Security Copilot Logins plugin, revising the documentation for both the Logins and Redact PII plugins, and removing the deprecated IP Geolocation documentation.

  • Added a new YAML manifest for the Security Copilot Logins plugin.
  • Updated documentation for Security Copilot Logins and Redact PII plugins.
  • Removed the IP Geolocation Readme, indicating deprecation.

Reviewed Changes

Copilot reviewed 7 out of 8 changed files in this pull request and generated 1 comment.

File Description
Plugins/Community Based Plugins/Copilot Logins/Manifest_SecurityCopilotLogins.yaml Provides configuration for the new Security Copilot Logins plugin
Plugins/Community Based Plugins/Copilot Logins/Readme.md Updates the plugin documentation with standardized sections and improved clarity
Plugins/Community Based Plugins/Redact PII/Readme.md Revises plugin description and documentation for redacting PII content
Plugins/Community Based Plugins/IP Geolocation/Readme.md Removes outdated documentation for the IP Geolocation plugin
Files not reviewed (1)
  • Plugins/Community Based Plugins/Copilot Logins/KQL_Plugin_CfSLogins.yaml: Language not supported
Comments suppressed due to low confidence (1)

Plugins/Community Based Plugins/Copilot Logins/Manifest_SecurityCopilotLogins.yaml:2

  • The plugin manifest uses 'SecurityCopilotLoginsStandalone' as the plugin name, but the skill definition later uses 'SecurityCopilotLogins'. Consider using consistent naming across the plugin configuration.
  Name: SecurityCopilotLoginsStandalone

Comment on lines +4 to 5
This plugin will redcat PII content from a session or a prompt ouput. It will also redact Entra ID object IDs theat may also be sensistive, for example GUIDs for an identity

Copy link
Preview

Copilot AI Mar 20, 2025

Choose a reason for hiding this comment

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

Multiple spelling errors detected: 'redcat' should be 'redact', 'ouput' should be 'output', 'theat' should be 'that', and 'sensistive' should be 'sensitive'.

Suggested change
This plugin will redcat PII content from a session or a prompt ouput. It will also redact Entra ID object IDs theat may also be sensistive, for example GUIDs for an identity
This plugin will redact PII content from a session or a prompt output. It will also redact Entra ID object IDs that may also be sensitive, for example GUIDs for an identity

Copilot is powered by AI, so mistakes are possible. Review output carefully before use.

Copy link
Collaborator

@KwachSean KwachSean left a comment

Choose a reason for hiding this comment

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

New Files Added:
Images:

Images/Community Plugins/RedactPII/RedactPIIPrompt.png
Images/Community Plugins/SecurityCopilotLogins/LoginsAndAnomalies.png
Plugin Manifests:

Plugins/Community Based Plugins/Copilot Logins/Manifest_SecurityCopilotLogins.yaml
Modified Files:
Plugins/Community Based Plugins/Copilot Logins/Readme.md

Title Change:
From: # Security Copilot Plugin: Security Copilot Portal Logins
To: # Security Copilot Login Activities
Updated Sections:
DESCRIPTION:
Added detailed description of the plugin functionality.
TYPE AND REQUIREMENTS:
Added type, source, and requirements.
SKILLS:
Added a table of skills with descriptions and parameters.
SAMPLE PROMPTS:
Added example prompt for using the plugin.
SCREENSHOTS:
Added a screenshot section with an image link.
Plugins/Community Based Plugins/Redact PII/Readme.md

Title Change:
From: # Security Copilot Plugin: Redact PII
To: # Redact Personally Identifiable Information
Updated Sections:
DESCRIPTION:
Added detailed description of the plugin functionality.
TYPE AND REQUIREMENTS:
Added type, source, and requirements.
SKILLS:
Added a table of skills with descriptions and parameters.
SAMPLE PROMPTS:
Added example prompt for using the plugin.
SCREENSHOTS:
Added a screenshot section with an image link.
Summary of Key Additions:
Images:
Two new images were added to the repository for visual representation of the plugins.
Plugin Manifests:
A new YAML file for the Copilot Logins plugin was added.
Readme Files:
Enhanced descriptions, requirements, and usage examples were added to the readme files for both the Copilot Logins and Redact PII plugins.

@KwachSean KwachSean requested a review from markrafn-msft March 24, 2025 08:23
@KwachSean KwachSean merged commit 202353c into Azure:main Mar 28, 2025
1 check passed
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