Skip to content
This repository is currently being migrated. It's locked while the migration is in progress.

LEAF-5792 - isSafeSqlIdentifier not accepting url with period#2912

Draft
shaneodd wants to merge 2 commits into
masterfrom
issue/LEAF-5792/url_with_period
Draft

LEAF-5792 - isSafeSqlIdentifier not accepting url with period#2912
shaneodd wants to merge 2 commits into
masterfrom
issue/LEAF-5792/url_with_period

Conversation

@shaneodd
Copy link
Copy Markdown
Contributor

Summary

add in period as an acceptable identifier

Impact

Issues with queries on sites with periods in the url

Testing

Go to a site with period in url, then attempt to add yourself to admin and view the admin area.

@github-actions
Copy link
Copy Markdown

Static Code Analysis Summary

Date: $(date)
Branch: 2912/merge
Commit: 279ebf8
Triggered by: pull_request

Analysis Results

PHP Analysis

  • PHPStan and Psalm reports available in artifacts

JavaScript/TypeScript Analysis

  • ESLint security and quality checks completed

Go Analysis

  • Gosec security scanning completed
  • Staticcheck analysis completed

HTML Analysis

  • HTMLHint validation completed

Security Scan

  • Semgrep multi-language security analysis completed
  • SARIF report uploaded to GitHub Security tab

Viewing Reports

All detailed reports are available as workflow artifacts and will be retained for 30 days.

Security findings can be viewed in the Security tab under Code scanning alerts.


Note: This scan is informational only and does not block merges.

@shaneodd shaneodd marked this pull request as draft March 16, 2026 20:33
@github-actions
Copy link
Copy Markdown

Static Code Analysis Summary

Date: $(date)
Branch: 2912/merge
Commit: 26970e4
Triggered by: pull_request

Analysis Results

PHP Analysis

  • PHPStan and Psalm reports available in artifacts

JavaScript/TypeScript Analysis

  • ESLint security and quality checks completed

Go Analysis

  • Gosec security scanning completed
  • Staticcheck analysis completed

HTML Analysis

  • HTMLHint validation completed

Security Scan

  • Semgrep multi-language security analysis completed
  • SARIF report uploaded to GitHub Security tab

Viewing Reports

All detailed reports are available as workflow artifacts and will be retained for 30 days.

Security findings can be viewed in the Security tab under Code scanning alerts.


Note: This scan is informational only and does not block merges.

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.

1 participant