Alonexc opened a new issue, #3084:
URL: https://github.com/apache/incubator-eventmesh/issues/3084

   ### Search before asking
   
   - [X] I had searched in the 
[issues](https://github.com/apache/eventmesh/issues?q=is%3Aissue) and found no 
similar issues.
   
   
   ### Enhancement Request
   
   
![image](https://user-images.githubusercontent.com/91315508/218020608-d3c1d7e4-27c5-4370-bcfb-ea6b01c0bb87.png)
   located at:
   
eventmesh-common/src/main/java/org/apache/eventmesh/common/config/FileLoad.java 
line 67
   analysis and explanation:
   1.Found a call to a method which will perform a byte to String (or String to 
byte) conversion, and will assume that the default platform encoding is 
suitable. This will cause the application behaviour to vary between platforms. 
Use an alternative API and specify a charset name or Charset object explicitly.
   2. The method creates an IO stream object, does not assign it to any fields, 
pass it to other methods that might close it, or return it, and does not appear 
to close the stream on all paths out of the method.  This may result in a file 
descriptor leak.  It is generally a good idea to use a finally block to ensure 
that streams are closed.
   
   
   ### Describe the solution you'd like
   
   e.g.
   
![image](https://user-images.githubusercontent.com/91315508/218020676-f5d278e9-a4f0-44ae-b12a-85a4b07aef9c.png)
   
   
   ### Are you willing to submit PR?
   
   - [ ] Yes I am willing to submit a PR!


-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to