Allow lowercase on logger change endpoint for MVC #7914

Closed
wants to merge 2 commits into
from

Projects

None yet

3 participants

@kazuki43zoo
Contributor

I've fixed to allow lowercase on logger change endpoint(POST /loggers/{name}) for Spring MVC.
Currently, IllegalArgumentException occur if contains lowercase into log level.

e.g.)

{
    "configuredLevel": "debug"
}

Please review this.

@kazuki43zoo kazuki43zoo Allow lowercase on logger change endpoint for MVC
64caf89
@snicoll
Member
snicoll commented Jan 9, 2017

I did fix that for the JMX endpoint and forgot to apply the same rule to the MVC counterpart. Good catch!

@snicoll snicoll self-assigned this Jan 9, 2017
@snicoll snicoll added this to the 1.5.0 milestone Jan 9, 2017
@kazuki43zoo kazuki43zoo Merge branch '1.5.x' into allow-lowercase-on-logger-endpoint
2833cfb
@snicoll snicoll added a commit that closed this pull request Jan 15, 2017
@snicoll snicoll Polish contribution
Closes gh-7914
f21c767
@snicoll snicoll closed this in f21c767 Jan 15, 2017
@snicoll
Member
snicoll commented Jan 15, 2017

merged, thanks!

@kazuki43zoo kazuki43zoo deleted the kazuki43zoo:allow-lowercase-on-logger-endpoint branch Jan 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment