steffow
commented
6 years ago In general yes. BUT: we need npms for this and the dependency checking can be quite painful. Need to balance here. We're already sourcing too many npms if you ask me.
Closed rhoerbe closed 6 years ago
steffow
commented
6 years ago In general yes. BUT: we need npms for this and the dependency checking can be quite painful. Need to balance here. We're already sourcing too many npms if you ask me.
saml_util.js creates an AuthnRequest using string concatenation with variable substitution. A safer way (for extensibility) would be to use an XML library.