8000 Support implementations that send invalid trailing 0x00 bytes in JSON messages by Neverminda · Pull Request #30 · enbility/ship-go · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Support implementations that send invalid trailing 0x00 bytes in JSON messages #30

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

Merged
merged 2 commits into from
Aug 17, 2024

Conversation

Neverminda
Copy link
Contributor
@Neverminda Neverminda commented Aug 17, 2024

Remove possible and erroneous trailing \x00 character. This fixes connection issues with the Porsche Mobile Charger Plus (PMCP)

Fixes #29

…t the end of JSON messages

remove possible trainling „\x00“ character this fixes connection issues with the Porsche Mobile Charger Plus
@DerAndereAndi
Copy link
Member

Danke @Neverminda für den PR, freue mich über die Unterstützung!

@DerAndereAndi DerAndereAndi changed the title Fix #29 Support implementations that send invalid trailing 0x00 bytes Support implementations that send invalid trailing 0x00 bytes in JSON messages Aug 17, 2024
@coveralls
Copy link
coveralls commented Aug 17, 2024

Coverage Status

coverage: 90.754% (+0.08%) from 90.672%
when pulling d309ded on Neverminda:fix#29
into d1bc62d on enbility:dev.

@DerAndereAndi
Copy link
Member

@Neverminda Wäre super wenn du die 2 Vorschläge noch integrieren könntest, dann mach ich das rein.

@Neverminda
Copy link
Contributor Author

Schon dabei, ich kämpfe gerade mit GitHub

@Neverminda
Copy link
Contributor Author

Ich hatte alles lokal gesquashed, das mag GitHub wohl nicht :)

@DerAndereAndi
Copy link
Member

Perfekt, vielen Dank. Mit nem Force-Push in den Branch hätte es auch funktionieren sollen. Aber so tut es auch :)

@DerAndereAndi DerAndereAndi merged commit 15af050 into enbility:dev Aug 17, 2024
3 checks passed
@Neverminda Neverminda deleted the fix#29 branch August 19, 2024 14:59
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.

Support implementations that send invalid 0x00 byte at the end of JSON messages
3 participants
0