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.

Sign up for GitHub

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

Jump to bottom

ELY-2379 Rename a local variable in FileAuditEndpoint class [local_variable_renamed] #2059

Closed
wants to merge 2 commits into from
Closed

ELY-2379 Rename a local variable in FileAuditEndpoint class [local_variable_renamed] #2059

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
6057ee8
local_variable_renamed
OluchukwuCatherineObi-Njoku Oct 26, 2023
2f74e5f
local variable renamed
OluchukwuCatherineObi-Njoku Oct 31, 2023
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
10 changes: 5 additions & 5 deletions audit/src/main/java/org/wildfly/security/audit/FileAuditEndpoint.java
View file
Open in desktop
Expand Up @@ -70,22 +70,22 @@ public class FileAuditEndpoint implements AuditEndpoint {
}

void setFile(final File file) throws IOException {
boolean ok = false;
boolean isFileSet = false;
final FileOutputStream fos = new FileOutputStream(file, true);
try {
final Writer writer = new OutputStreamWriter(new BufferedOutputStream(fos), this.charset);
try {
this.fileDescriptor = fos.getFD();
this.writer = writer;
this.file = file;
ok = true;
isFileSet = true;
} finally {
if (! ok) {
if (! isFileSet) {
safeClose(writer);
}
}
} finally {
if (! ok) {
if (! isFileSet) {
safeClose(fos);
}
}
Expand Down Expand Up @@ -121,7 +121,7 @@ void write(String toWrite) throws IOException {
* This method is NO-OP by default. It is intended to be overridden by subclasses
* which need to perform some operation before every writing into the target local file.
*
* This method can be invoked only in synchronization block surrounding one log message processing.
* This method can be invisFileSeted only in synchronization block surrounding one log message processing.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This shouldn't be renamed.

*
* @param instant time of the message acceptance
*/
Expand Down