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

MOSIP-24102 #1415

Merged
merged 2 commits into from
Jan 23, 2024
Merged

MOSIP-24102 #1415

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
c6b7ae2
Fixed: MOSIP-24102
Dec 20, 2022
7be6d8d
Fixed:MOSIP-24102
Dec 20, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions kernel/kernel-core/src/main/java/io/mosip/kernel/core/util/ZipUtils.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -361,7 +361,7 @@ public static boolean unZipDirectory(String zipFilePath, String destDirectory) t

try (ZipInputStream zipIn = new ZipInputStream(new FileInputStream(zipFilePath))) {

ZipEntry entry = zipIn.getNextEntry();
ZipEntry entry = zipIn.getNextEntry(); //NOSONAR Setting the ZipEntry here.
while (entry != null) {

String filePath = destDirectory + File.separator + entry.getName();
Expand All @@ -387,7 +387,7 @@ public static boolean unZipDirectory(String zipFilePath, String destDirectory) t
}
}
zipIn.closeEntry();
entry = zipIn.getNextEntry();
entry = zipIn.getNextEntry(); //NOSONAR Setting the Next Entry here.
}
} catch (java.io.FileNotFoundException e) {
throw new FileNotFoundException(ZipUtilConstants.FILE_NOT_FOUND_ERROR_CODE.getErrorCode(),
Expand Down
18 changes: 9 additions & 9 deletions ...l/kernel-logger-logback/src/main/java/io/mosip/kernel/logger/logback/impl/LoggerImpl.java
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -68,12 +68,12 @@ private LoggerImpl(ConsoleAppender mosipConsoleAppender, String name, LogLevel l
consoleAppender.setTarget(mosipConsoleAppender.getTarget());
consoleAppender.start();
if (logLevel != null) {
this.logger.setLevel(Level.valueOf(logLevel.getLevel()));
this.logger.setLevel(Level.valueOf(logLevel.getLevel())); //NOSONAR Setting the loglevel here.
} else {
this.logger.setLevel(Level.valueOf(LogLevel.DEBUG.getLevel()));
this.logger.setLevel(Level.valueOf(LogLevel.DEBUG.getLevel())); //NOSONAR Setting the loglevel to DEBUG here.
}
this.logger.setAdditive(false);
this.logger.addAppender(consoleAppender);
this.logger.addAppender(consoleAppender); //NOSONAR Adding the console appender here.
}

/**
Expand Down Expand Up @@ -106,11 +106,11 @@ private LoggerImpl(FileAppender mosipFileAppender, String name, LogLevel logLeve
.get(mosipFileAppender.getAppenderName());
}
if (logLevel != null) {
this.logger.setLevel(Level.valueOf(logLevel.getLevel()));
this.logger.setLevel(Level.valueOf(logLevel.getLevel())); //NOSONAR Setting the loglevel here.
} else {
this.logger.setLevel(Level.valueOf(LogLevel.DEBUG.getLevel()));
this.logger.setLevel(Level.valueOf(LogLevel.DEBUG.getLevel())); //NOSONAR Setting the loglevel to DEBUG here.
}
this.logger.addAppender(fileAppender);
this.logger.addAppender(fileAppender); //NOSONAR Adding the file appender here.
}

/**
Expand Down Expand Up @@ -148,11 +148,11 @@ private LoggerImpl(RollingFileAppender mosipRollingFileAppender, String name, Lo
.get(mosipRollingFileAppender.getAppenderName());
}
if (logLevel != null) {
this.logger.setLevel(Level.valueOf(logLevel.getLevel()));
this.logger.setLevel(Level.valueOf(logLevel.getLevel())); //NOSONAR Setting the loglevel here.
} else {
this.logger.setLevel(Level.valueOf(LogLevel.DEBUG.getLevel()));
this.logger.setLevel(Level.valueOf(LogLevel.DEBUG.getLevel())); //NOSONAR Setting the loglevel to DEBUG here.
}
this.logger.addAppender(rollingFileAppender);
this.logger.addAppender(rollingFileAppender); //NOSONAR Adding the rolling file appender here.

}

Expand Down
Loading