Skip to content

Add support for skipping TLS verification to Harper#164

Merged
cap10morgan merged 1 commit intomainfrom
feat/tls-skip-verify
Jan 28, 2026
Merged

Add support for skipping TLS verification to Harper#164
cap10morgan merged 1 commit intomainfrom
feat/tls-skip-verify

Conversation

@cap10morgan
Copy link
Contributor

This is useful for dev against hostname-based clusters which require an HTTPS connection but typically use self-signed certs.

This is useful for dev against hostname-based clusters which require an HTTPS connection but typically use self-signed certs
@socket-security
Copy link

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addednpm/​@​grafana/​plugin-ui@​0.12.1951008592100

View full report

@socket-security
Copy link

Warning

Review the following alerts detected in dependencies.

According to your organization's Security Policy, it is recommended to resolve "Warn" alerts. Learn more about Socket for GitHub.

Action Severity Alert  (click "▶" to expand/collapse)
Warn Medium
Deprecated by its maintainer: npm @babel/polyfill

Reason: 🚨 This package has been deprecated in favor of separate inclusion of a polyfill and regenerator-runtime (when needed). See the @babel/polyfill docs (https://babeljs.io/docs/en/babel-polyfill) for more information.

From: package-lock.jsonnpm/@grafana/plugin-ui@0.12.1npm/@babel/polyfill@7.12.1

ℹ Read more on: This package | This alert | What is a deprecated package?

Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support@socket.dev.

Suggestion: Research the state of the package and determine if there are non-deprecated versions that can be used, or if it should be replaced with a new, supported solution.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore npm/@babel/polyfill@7.12.1. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

Warn Medium
Deprecated by its maintainer: npm core-js

Reason: core-js@<3.23.3 is no longer maintained and not recommended for usage due to the number of issues. Because of the V8 engine whims, feature detection in old core-js versions could cause a slowdown up to 100x even if nothing is polyfilled. Some versions have web compatibility issues. Please, upgrade your dependencies to the actual version of core-js.

From: package-lock.jsonnpm/@grafana/plugin-ui@0.12.1npm/core-js@2.6.12

ℹ Read more on: This package | This alert | What is a deprecated package?

Next steps: Take a moment to review the security alert above. Review the linked package source code to understand the potential risk. Ensure the package is not malicious before proceeding. If you're unsure how to proceed, reach out to your security team or ask the Socket team for help at support@socket.dev.

Suggestion: Research the state of the package and determine if there are non-deprecated versions that can be used, or if it should be replaced with a new, supported solution.

Mark the package as acceptable risk. To ignore this alert only in this pull request, reply with the comment @SocketSecurity ignore npm/core-js@2.6.12. You can also ignore all packages with @SocketSecurity ignore-all. To ignore an alert for all future pull requests, use Socket's Dashboard to change the triage state of this alert.

View full report

@github-actions
Copy link
Contributor

Bundle Size Changes

Hello! 👋 This comment was generated by a Github Action to help you and reviewers understand the impact of your PR on frontend bundle sizes.

Whenever this PR is updated, this comment will update to reflect the latest changes.

EntryPoint Size % Diff
module 17.19 KB (+3.04 KB) +21.50%
Files Total bundle size % Diff
9 1000.34 KB (+3.04 KB) +0.31%
View detailed bundle information

Added

No assets were added

Removed

No assets were removed

Bigger

Name Size % Diff
module.js 17.19 KB (+3.04 KB) +21.50%

Smaller

No assets were smaller

View module information

Added

Name Size % Diff
@grafana/plugin-ui 3.67 KB (+3.67 KB) -
react 1.05 KB (+1.05 KB) -

Removed

No modules were removed

Bigger

Name Size % Diff
./components/ConfigEditor.tsx 5.74 KB (+1.06 KB) +22.53%

Smaller

No modules were smaller

@cap10morgan cap10morgan requested a review from a team January 27, 2026 23:04
@cap10morgan
Copy link
Contributor Author

Re: the Socket warnings, the dependencies in here are mostly defined by the Grafana plugin template. But changes to it do get synced up here eventually. So if they decide to drop those deprecated dependencies, that change should make its way here. But I don't think we want to preemptively remove them.

@cap10morgan cap10morgan merged commit 6d275fd into main Jan 28, 2026
16 checks passed
@cap10morgan cap10morgan deleted the feat/tls-skip-verify branch January 28, 2026 17:17
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