palantir / conjure-java

Conjure generator for Java clients and servers
Apache License 2.0
27 stars 43 forks source link

nullable PlainSerDe#deserializeRid input #2216

Closed tpetracca closed 6 months ago

tpetracca commented 6 months ago

already true of the implementation https://github.com/palantir/conjure-java/blob/develop/conjure-java-undertow-runtime/src/main/java/com/palantir/conjure/java/undertow/runtime/ConjurePlainSerDe.java#L324

    @Override
    public ResourceIdentifier deserializeRid(@Nullable String in) {
        checkArgumentNotNull(in);
        try {
            return ResourceIdentifier.valueOf(in);
        } catch (RuntimeException ex) {
            throw new SafeIllegalArgumentException("failed to deserialize rid", ex);
        }
    }

==COMMIT_MSG== nullable PlainSerDe#deserializeRid input ==COMMIT_MSG==

Possible downsides?

svc-autorelease commented 6 months ago

Released 8.10.0