bito-code-review[bot] commented on PR #15976:
URL: https://github.com/apache/grails-core/pull/15976#issuecomment-4949448709
<!-- Bito Reply -->
Yes, that is a good suggestion. Making the encoding configurable while
defaulting to `GrailsWebUtil.DEFAULT_ENCODING` would provide more flexibility
for different environments while maintaining a sensible default. You could
update the class to allow setting the encoding via a property or constructor,
ensuring it falls back to the default if not explicitly provided.
**grails-rest-transforms/src/main/groovy/org/grails/plugins/web/rest/render/html/DefaultHtmlRenderer.groovy**
```
String encoding = GrailsWebUtil.DEFAULT_ENCODING
DefaultHtmlRenderer(Class<T> targetType, String encoding =
GrailsWebUtil.DEFAULT_ENCODING) {
this.targetType = targetType
this.encoding = encoding
}
```
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
