ocpp icon indicating copy to clipboard operation
ocpp copied to clipboard
verified publisher icon mobilityhouse

Handle responder_url edge case

Open proelke opened this issue 3 years ago • 1 comments

In the 2.0.1 spec for AuthorizeRequest and GetCertificateStatusRequest there is a field responderURL that does not get converted from snake_case responder_url to camelCase responderURL. This PR adds handling of this edge case. in SetNetworkProfileRequest there is a field ocppCsmsUrl which is handled correctly but causes problems for a more general solution.

Let me know if the conditionals work for you.

proelke avatar May 11 '22 15:05 proelke

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

sonarqubecloud[bot] avatar May 11 '22 15:05 sonarqubecloud[bot]

@Jared-Newell-Mobility This one is also required for a v1 release. You can take it over the issue and finish it.

OrangeTux avatar Jan 26 '24 12:01 OrangeTux

Thanks for highlighting this, it has been replaced with PR https://github.com/mobilityhouse/ocpp/pull/592, so it will close for now

Jared-Newell-Mobility avatar Feb 05 '24 10:02 Jared-Newell-Mobility