Skip to content

fix: remove Get-WiFiKnownPasswords function to prevent false positives #49

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

Conversation

danstis
Copy link
Owner

@danstis danstis commented Jul 29, 2025

User description

Fixes #48


PR Type

Bug fix


Description

  • Remove Get-WiFiKnownPasswords function file

  • Exclude function in module manifest exports

  • Remove function import from module script


Diagram Walkthrough

flowchart LR
  PSDF["PSF.psd1"]
  PSM["PSF.psm1"]
  FnFile["Get-WiFiKnownPasswords.ps1"]
  PSDF -- "Remove from FunctionsToExport" --> PSM
  PSM -- "Remove import line" --> FnFile
  FnFile -- "Delete file" --> Removed["Function removed"]
Loading

File Walkthrough

Relevant files
Configuration changes
PSF.psd1
Update FunctionsToExport list                                                       

PSF.psd1

  • Removed Get-WiFiKnownPasswords from FunctionsToExport
+1/-1     
PSF.psm1
Remove function import                                                                     

PSF.psm1

  • Removed import of Get-WiFiKnownPasswords.ps1
+0/-1     
Bug fix
Get-WiFiKnownPasswords.ps1
Remove WiFi function file                                                               

functions/Get-WiFiKnownPasswords.ps1

  • Deleted entire Get-WiFiKnownPasswords function file
+0/-32   

Copy link

gitguardian bot commented Jul 30, 2025

⚠️ GitGuardian has uncovered 1 secret following the scan of your pull request.

Please consider investigating the findings and remediating the incidents. Failure to do so may lead to compromising the associated services or software components.

🔎 Detected hardcoded secret in your pull request
GitGuardian id GitGuardian status Secret Commit Filename
19429967 Triggered Generic Password 4cf01fa PSF.psd1 View secret
🛠 Guidelines to remediate hardcoded secrets
  1. Understand the implications of revoking this secret by investigating where it is used in your code.
  2. Replace and store your secret safely. Learn here the best practices.
  3. Revoke and rotate this secret.
  4. If possible, rewrite git history. Rewriting git history is not a trivial act. You might completely break other contributing developers' workflow and you risk accidentally deleting legitimate data.

To avoid such incidents in the future consider


🦉 GitGuardian detects secrets in your source code to help developers and security teams secure the modern development process. You are seeing this because you or someone else with access to this repository has authorized GitGuardian to scan your pull request.

@sophie-syntax sophie-syntax bot changed the title Remove Get-WiFiKnownPasswords function due to security scanner false positives fix: remove Get-WiFiKnownPasswords function to prevent false positives Jul 30, 2025
Copy link

sophie-syntax bot commented Jul 30, 2025

PR Code Suggestions ✨

No code suggestions found for the PR.

@danstis danstis merged commit 308856b into master Jul 30, 2025
3 of 4 checks passed
@danstis danstis deleted the danstis/48-Remove-Get-WiFiKnownPasswords-function-due-to-security-scanner-false-positives branch July 30, 2025 00:03
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.

Remove Get-WiFiKnownPasswords function due to security scanner false positives
1 participant
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy