Skip to content
Navigation Menu
Sign in
Appearance settings
Platform
AI CODE CREATION
GitHub Copilot
Write better code with AI
GitHub Copilot app
Direct agents from issue to merge
MCP Registry
Integrate external tools
DEVELOPER WORKFLOWS
Actions
Automate any workflow
Codespaces
Instant dev environments
Issues
Plan and track work
Code Review
Manage code changes
Code Quality
Enforce quality at merge
APPLICATION SECURITY
GitHub Advanced Security
Find and fix vulnerabilities
Code security
Secure your code as you build
Secret protection
Stop leaks before they start
EXPLORE
Why GitHub
Documentation
Blog
Changelog
Marketplace
View all features
Solutions
BY COMPANY SIZE
Enterprises
Small and medium teams
Startups
Nonprofits
BY USE CASE
App Modernization
DevSecOps
DevOps
CI/CD
View all use cases
BY INDUSTRY
Healthcare
Financial services
Manufacturing
Government
View all industries
View all solutions
Resources
EXPLORE BY TOPIC
AI
Software Development
DevOps
Security
View all topics
EXPLORE BY TYPE
Customer stories
Events & webinars
Ebooks & reports
Business insights
GitHub Skills
SUPPORT & SERVICES
Documentation
Customer support
Community forum
Trust center
Partners
View all resources
Open Source
COMMUNITY
GitHub Sponsors
Fund open source developers
PROGRAMS
Security Lab
Maintainer Community
GitHub Stars
Archive Program
REPOSITORIES
Topics
Trending
Collections
Enterprise
ENTERPRISE SOLUTIONS
Enterprise platform
AI-powered developer platform
AVAILABLE ADD-ONS
GitHub Advanced Security
Enterprise-grade security features
Copilot for Business
Enterprise-grade AI features
Premium Support
Enterprise-grade 24/7 support
Pricing
Search
/
Sign in
Sign up
Appearance settings
You signed in with another tab or window.
Reload
to refresh your session.
You signed out in another tab or window.
Reload
to refresh your session.
You switched accounts on another tab or window.
Reload
to refresh your session.
Dismiss alert
{{ message }}
node-saml
/
xml-crypto
Public
Notifications
You must be signed in to change notification settings
Fork
190
Star
212
Code
Issues
38
Pull requests
22
Discussions
Actions
Projects
Wiki
Security and quality
4
Insights
Additional navigation options
Code
Issues
Pull requests
Discussions
Actions
Projects
Wiki
Security and quality
Insights
fix: omit <Transforms> element when no transforms are specified
- #542
#542
Open
msheby
wants to merge 25 commits into
node-saml:master
node-saml/xml-crypto:master
from
msheby:fix/omit-empty-transforms-clean
msheby/xml-crypto:fix/omit-empty-transforms-clean
Copy head branch name to clipboard
Conversation
Commits
25
(25)
Checks
Files changed
Open
fix: omit <Transforms> element when no transforms are specified
#542
msheby
wants to merge 25 commits into
node-saml:master
node-saml/xml-crypto:master
from
msheby:fix/omit-empty-transforms-clean
msheby/xml-crypto:fix/omit-empty-transforms-clean
Copy head branch name to clipboard
Commits
Commits on Aug 11, 2026
fix: omit <Transforms> element when no transforms are specified
Show description for 7e30184
Matthew Sheby
committed
7e30184
View commit details
Copy full SHA for 7e30184
Browse repository at this point
test: replace non-null assertions with deep.equal on ref.transforms
Show description for e33343c
Matthew Sheby
committed
e33343c
View commit details
Copy full SHA for e33343c
Browse repository at this point
Commits on Aug 12, 2026
test: cover both omitted and empty transforms in Transforms-omission test
Show description for 21df3b8
Matthew Sheby
committed
21df3b8
View commit details
Copy full SHA for 21df3b8
Browse repository at this point
fix: derive ancestor namespaces from the referenced node, not re-running xpath
Show description for c5d1976
Matthew Sheby
committed
c5d1976
View commit details
Copy full SHA for c5d1976
Browse repository at this point
Commits on Aug 17, 2026
docs: expand buildAncestorNsForElement JSDoc and fix prettier formatting
Matthew Sheby
committed
3a2a63c
View commit details
Copy full SHA for 3a2a63c
Browse repository at this point
Commits on Sep 8, 2026
Merge branch 'master' into fix/omit-empty-transforms-clean
cjbarth
authored
ebd058c
View commit details
Copy full SHA for ebd058c
Browse repository at this point
fix: guard against undefined transforms in enveloped-signature check
Matthew Sheby
committed
9e24d28
View commit details
Copy full SHA for 9e24d28
Browse repository at this point
test: exercise addAllReferences multi-match in ancestor-namespace regression test
Matthew Sheby
committed
82bc488
View commit details
Copy full SHA for 82bc488
Browse repository at this point
Merge branch 'master' into pr/542
Show description for 1834d27
cjbarth
and
claude
committed
1834d27
View commit details
Copy full SHA for 1834d27
Browse repository at this point
fix: keep Reference.transforms required and tighten review follow-ups
Show description for 6e16874
cjbarth
and
claude
committed
6e16874
View commit details
Copy full SHA for 6e16874
Browse repository at this point
refactor: fold findAncestorNsForNode wrapper into its helper
Show description for 6a7ac33
cjbarth
and
claude
committed
6a7ac33
View commit details
Copy full SHA for 6a7ac33
Browse repository at this point
refactor: replace `export *` in index.ts with explicit re-export lists
Show description for 6920712
cjbarth
and
claude
committed
6920712
View commit details
Copy full SHA for 6920712
Browse repository at this point
Merge branch 'node-saml:master' into fix/omit-empty-transforms-clean
msheby
authored
bcc7e78
View commit details
Copy full SHA for bcc7e78
Browse repository at this point
test: cover the non-element document subset guard in findAncestorNs
Show description for c6bb8c0
cjbarth
and
claude
committed
c6bb8c0
View commit details
Copy full SHA for c6bb8c0
Browse repository at this point
Lint
cjbarth
committed
a0b3e14
View commit details
Copy full SHA for a0b3e14
Browse repository at this point
Commits on Sep 9, 2026
fix: assert element-ness in getCanonReferenceXml instead of silently skipping
Show description for 6395317
cjbarth
and
claude
committed
6395317
View commit details
Copy full SHA for 6395317
Browse repository at this point
test: assert refusal rather than a dependency's error wording
Show description for 1efc2c1
cjbarth
and
claude
committed
1efc2c1
View commit details
Copy full SHA for 1efc2c1
Browse repository at this point
fix: reject signing a reference that encloses the signature without a transform
Show description for ce8d32a
cjbarth
and
claude
committed
ce8d32a
View commit details
Copy full SHA for ce8d32a
Browse repository at this point
refactor: let algorithms declare whether they drop nodes
Show description for abfa1ad
cjbarth
and
claude
committed
abfa1ad
View commit details
Copy full SHA for abfa1ad
Browse repository at this point
fix!: require removesNodes on CanonicalizationOrTransformationAlgorithm
Show description for a752144
cjbarth
and
claude
committed
a752144
View commit details
Copy full SHA for a752144
Browse repository at this point
fix: enforce removesNodes at runtime and let subclasses declare it
Show description for f7d916d
cjbarth
and
claude
committed
f7d916d
View commit details
Copy full SHA for f7d916d
Browse repository at this point
fix: validate removesNodes in findCanonicalizationAlgorithm
Show description for 7514d6d
cjbarth
and
claude
committed
7514d6d
View commit details
Copy full SHA for 7514d6d
Browse repository at this point
docs: state that removesNodes is enforced on verification too
Show description for dcc89d4
cjbarth
and
claude
committed
dcc89d4
View commit details
Copy full SHA for dcc89d4
Browse repository at this point
Commits on Sep 10, 2026
Merge branch 'master' into fix/omit-empty-transforms-clean
Show description for 1b9ce75
Matthew Sheby
committed
1b9ce75
View commit details
Copy full SHA for 1b9ce75
Browse repository at this point
Commits on Sep 11, 2026
fix: drop unreachable nullish fallback on reference transforms
Show description for faab399
Matthew Sheby
committed
faab399
View commit details
Copy full SHA for faab399
Browse repository at this point
You can’t perform that action at this time.