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

ConfigurationStrSubstitutor - incorrect class-level Javadoc #3170

Open
JWT007 opened this issue Nov 4, 2024 · 1 comment
Open

ConfigurationStrSubstitutor - incorrect class-level Javadoc #3170

JWT007 opened this issue Nov 4, 2024 · 1 comment
Assignees

Comments

@JWT007
Copy link
Contributor

JWT007 commented Nov 4, 2024

ConfigurationStrSubstitutor (Log4j 2.24.1)

Small Javadoc inconsistency - probably copy/paste error?

In the ConfigurationStrSubstitutor javadoc, the RuntimeStrStubstitutoris incorrectly referenced.

/**
 * {@link RuntimeStrSubstitutor} is a {@link StrSubstitutor} which only supports recursive evaluation of lookups.
 * This can be dangerous when combined with user-provided inputs, and should only be used on data directly from
 * a configuration.
 */
public final class ConfigurationStrSubstitutor extends StrSubstitutor {
@JWT007 JWT007 changed the title ConfigurationStrSubstitutor - incorrect class-levell Javadoc ConfigurationStrSubstitutor - incorrect class-level Javadoc Nov 4, 2024
@jaykataria1111
Copy link

I will upload a PR for this soon, small fix. does not need that much investigation. cc: @ppkarwasz

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

No branches or pull requests

3 participants