Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updates xml crypto to 2.0.0 in version 1.x-latest #305

Closed
wants to merge 129 commits into from
Closed

Updates xml crypto to 2.0.0 in version 1.x-latest #305

wants to merge 129 commits into from

Conversation

marcosins
Copy link

@marcosins marcosins commented Dec 30, 2020

Description

Related issues

Checklist

bhoriuchi and others added 30 commits November 22, 2016 07:46
objects would not have been parsed
handle the case where the header for set-cookie is a string and not an array
changed double to single quotes in tests
JSON objects might provide properties in different order than the XML
elements. The fix addresses the issue.

This PR is a new implementation of #99
fix: make sure xml elements are mapped based the order of xsd
 * fix: upgrade deps (Raymond Feng)
 * fix: make sure xml elements are mapped based the order of xsd (Raymond Feng)
 * Fix Typo: RPC Literal (Thomas Richter)
 * check the original passwordDigest method fails against SoapUI output (Tom Hodder)
 * revert formatting changes by format on save in vscode (Tom Hodder)
 * Updated passwordDigest method and related tests (Tom Hodder)
 * missing semi colons (Branden Horiuchi)
 * removed lodash dependency (Branden Horiuchi)
 * added CookieSecurity test (Branden Horiuchi)
 * better cookie parsing handle the case where the header for set-cookie is a string and not an array (Branden Horiuchi)
 * fixing header parse objects would not have been parsed (Branden Horiuchi)
 * added cookie security (Branden Horiuchi)
 * fix: improve wsdl types processing for multiple schemas (Raymond Feng)
 * chore: clean up deps (Raymond Feng)
 * fix: add array check for concat (Raymond Feng)
 * build: drop node 4.x support (Raymond Feng)
fix xsd include/import recursive processing
 * fix xsd include/import recursive processing (Raymond Feng)
raymondfeng and others added 18 commits July 19, 2019 09:37
 * add support attachments (Allan Marques)
Avoid line feed problems (Win/Linux)
fixed formating in README
Update `.npmrc` to avoid shipping garbage files to consumers on npm

The largest folder is `coverage`, sitting at 1.8MB.

```
$ du -h -d1 strong-soap/  | sort -h
0       strong-soap/node_modules
4.0K    strong-soap/.vscode
8.0K    strong-soap/.github
60K     strong-soap/example
72K     strong-soap/xsds
120K    strong-soap/intl
432K    strong-soap/lib
440K    strong-soap/src
476K    strong-soap/.nyc_output
1.8M    strong-soap/coverage
3.5M    strong-soap/
```
Remove unnecessary files from npm packages
 * chore: add package-lock.json (Raymond Feng)
 * Remove unnecessary files from npm packages (Avindra Goolcharan)
 * fixed formating in README (Christopher Wade)
 * chore: add stalebot (Diana Lau)
 * Avoid line feed problems (Win/Linux) (Christian Herzog)
 * Allow nillable date to be null (Richard Sunbury)
 * chore: remove unused require statements (Raymond Feng)
@marcosins
Copy link
Author

I had to update 1.x-latest before applying my changes since the branch has only commits until 1.4.2 and i'm submitting this against 1.22.1.

@marcosins
Copy link
Author

Commit linter errors are for old commits not made by me. What should I do with them?

**  Linting commit logs
**
**  15 problems found:
**    b159f53b32 - Remove custom registry url from `package-lock.json`: First line should be 50 characters or less (saw 51)
**    09c894aa0c - 1.6.0: line 3 longer than 72 characters.
**    2000876981 - fix: improve wsdl types processing for multiple sch: First line should be 50 characters or less (saw 55)
**    6609d1e924 - 1.5.0: line 4 longer than 72 characters.
**    6609d1e924 - 1.5.0: line 6 longer than 72 characters.
**    6609d1e924 - 1.5.0: line 12 longer than 72 characters.
**    6609d1e924 - 1.5.0: line 13 longer than 72 characters.
**    eba0a9459a - Merge branch 'master' of https://github.com/limepep: First line should be 50 characters or less (saw 89)
**    3338eb342e - fix: make sure xml elements are mapped based the or: First line should be 50 characters or less (saw 61)
**    3338eb342e - fix: make sure xml elements are mapped based the or: line 6 longer than 72 characters.
**    bec3df0e6c - Merge branch 'master' of https://github.com/bhoriuc: First line should be 50 characters or less (saw 87)
**    c638518749 - check the original passwordDigest method fails agai: First line should be 50 characters or less (saw 68)
**    aa2ec34f02 - revert formatting changes by format on save in vsco: First line should be 50 characters or less (saw 53)
**    362da75f53 - better cookie parsing: Second line should be empty.<handle the case where the header for set-cookie is a string and not an array>
**    d9e9a01cbb - fixing header parse: Second line should be empty.<objects would not have been parsed>
**

@marcosins
Copy link
Author

I would like to keep working in this PR but I need someone from the team help me.

@stale
Copy link

stale bot commented Jul 14, 2021

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Jul 14, 2021
@stale
Copy link

stale bot commented Jul 28, 2021

This issue has been closed due to continued inactivity. Thank you for your understanding. If you believe this to be in error, please contact one of the code owners, listed in the CODEOWNERS file at the top-level of this repository.

@stale stale bot closed this Jul 28, 2021
@marcosins marcosins deleted the 1.x-latest branch June 1, 2022 02:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet