Skip to content

Fix TLS Decoder#301

Merged
spring1843 merged 6 commits into
mainfrom
cert-decrypt
Jun 7, 2026
Merged

Fix TLS Decoder#301
spring1843 merged 6 commits into
mainfrom
cert-decrypt

Conversation

@spring1843
Copy link
Copy Markdown
Owner

  • Fix cert decoder to use real decoder rather than mock data

Copilot AI review requested due to automatic review settings June 7, 2026 02:53
@netlify
Copy link
Copy Markdown

netlify Bot commented Jun 7, 2026

Deploy Preview for freedevtool ready!

Name Link
🔨 Latest commit 29849de
🔍 Latest deploy log https://app.netlify.com/projects/freedevtool/deploys/6a24e65dd25c410008208435
😎 Deploy Preview https://deploy-preview-301--freedevtool.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

Copy link
Copy Markdown
Contributor

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 replaces the TLS Decoder tool’s mock certificate parsing with real X.509 parsing via @peculiar/x509, updates the bundled default certificate example, and adjusts E2E assertions to account for CodeMirror rendering only visible lines.

Changes:

  • Implement real certificate parsing, extension rendering, fingerprints, and SAN listing in the TLS Decoder UI.
  • Replace the TLS decoder default certificate with a real-world example PEM.
  • Update TLS Decoder E2E tests to validate only the visible (initial) portion of the default certificate.

Reviewed changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 5 comments.

Show a summary per file
File Description
client/src/pages/tools/tls-decoder.tsx Switches from mock parsing to real X.509 decoding and renders detailed extension/fingerprint/SAN info.
client/src/data/defaults.ts Updates the default TLS decoder PEM example.
tests/e2e/tools/tls-decoder.spec.ts Adjusts default-input checks to match CodeMirror’s visible-line rendering behavior.
package.json Adds @peculiar/x509 and reflect-metadata dependencies needed for real parsing.
package-lock.json Locks new dependency tree for X.509 parsing support.

Comment thread client/src/pages/tools/tls-decoder.tsx
Comment thread client/src/pages/tools/tls-decoder.tsx
Comment thread client/src/pages/tools/tls-decoder.tsx
Comment thread client/src/data/defaults.ts Outdated
Comment thread client/src/pages/tools/tls-decoder.tsx Outdated
spring1843 and others added 5 commits June 6, 2026 20:11
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@spring1843 spring1843 merged commit 125e810 into main Jun 7, 2026
5 checks passed
@spring1843 spring1843 deleted the cert-decrypt branch June 7, 2026 03:40
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