Allow PemContent.of to work with untrimmed content

[yingzhuo]

Hello.
I am currently using Spring Boot 3.3.1, I noticed the utility org.springframework.boot.ssl.pem.PemContent is very useful.

how about add a PropertyEditor to the package? here is my code:

package org.springframework.boot.ssl.pem;

import org.springframework.boot.ssl.pem.PemContent;
import org.springframework.util.StringUtils;

import java.beans.PropertyEditorSupport;
import java.util.Arrays;
import java.util.stream.Collectors;

/**
 * @author Ying Zhuo
 */
public class PemContentEditor extends PropertyEditorSupport {

    /**
     * {@inheritDoc}
     */
    @Override
    public final void setAsText(String text) throws IllegalArgumentException {
        try {
            super.setValue(PemContent.of(trimContent(text)));
        } catch (Exception e) {
            throw new IllegalArgumentException(e.getMessage(), e);
        }
    }

    private String trimContent(String text) {
        return Arrays.stream(StringUtils.delimitedListToStringArray(text, "\n"))
                .map(String::trim)
                .collect(Collectors.joining("\n"));
    }
}

so i can config my bean like this:

<bean id="jwtSigner" class="com.mycompany.myproject.jwt.signer.AsymmetricSignerFactoryBean">
    <property name="certificateContent">
        <value>
            <![CDATA[
            -----BEGIN CERTIFICATE-----
            x509 format content here
            -----END CERTIFICATE-----
            ]]>
        </value>
    </property>
    <property name="keyContent">
        <value>
            <![CDATA[
            -----BEGIN PRIVATE KEY-----
            PKCS#8 format content here
            -----END PRIVATE KEY-----
            ]]>
        </value>
    </property>
    <property name="keyPassword">
        <null/>
    </property>
</bean>

@scottfrederick your code of SSL is very cool, thanks.

[philwebb]

I'm not to keen to add a PropertyEditor, but a Converter might be an option. I wonder if ObjectToObjectConverter might work out-of-the-box if we refined the of method to deal with trimming strings.

[philwebb]

We discussed this today and we don't think we should add the property editor, but we do think we can make the of() method do the trimming.

[yingzhuo]

We discussed this today and we don't think we should add the property editor, but we do think we can make the of() method do the trimming.

Thank you, sir. I am so glad to hear that.

[wilkinsona]

If we refine only of, I wonder if isPresentInText(String) and the various load methods behaving differently will cause confusion. It feels like trimming should either be done consistently or it should only be done by a method that makes it clear that its behavior is different. Perhaps of could be that method but it feels a bit hidden and might be better if the method name made the behavior clear.

[yingzhuo]

Perhaps of could be that method but it feels a bit hidden and might be better if the method name made the behavior clear.

just a hint, sir.

ObjectToObjectConverter can only find static method valueOf() or of(). if method name be 'ofTrimmedString()'
the converter cannot work.

[mhalbritter]

Is the trimming necessary? the getCertificates and getPrivateKey methods also work with un-trimmed content (they ignore everything before the --- BEGIN header and the footer and don't care about whitespace in between).

The only way to get the original un-trimmed content back is by calling toString.

So maybe just adding a converter without the trimming is sufficient?

@yingzhuo did i miss something? Is the trimming necessary?