vanitasvitae/Smack
1
0
Fork 0
mirror of https://github.com/vanitasvitae/Smack.git synced 2024-12-22 18:48:00 +01:00
Code Issues Releases Wiki Activity

Conditionally reduce severity of roster reload error logging

Browse source 
If the roster feature is not supported by the server, there's no need to
log this at SEVERE log level.
This commit is contained in:
Danny Baumann 2022-02-07 14:36:45 +01:00
parent 4a83d2957e
commit aafc24a966
1 changed files with 6 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

9
smack-im/src/main/java/org/jivesoftware/smack/roster/Roster.java
View file

@ -469,11 +469,14 @@ public final class Roster extends Manager {
@Override @Override
public void processException(Exception exception) { public void processException(Exception exception) {
rosterState = RosterState.uninitialized; rosterState = RosterState.uninitialized;
Level logLevel; Level logLevel = Level.SEVERE;
if (exception instanceof NotConnectedException) { if (exception instanceof NotConnectedException) {
logLevel = Level.FINE; logLevel = Level.FINE;
} else { } else if (exception instanceof XMPPErrorException) {
logLevel = Level.SEVERE; Condition condition = ((XMPPErrorException) exception).getStanzaError().getCondition();
if (condition == Condition.feature_not_implemented || condition == Condition.service_unavailable) {
logLevel = Level.FINE;
}
} }
LOGGER.log(logLevel, "Exception reloading roster", exception); LOGGER.log(logLevel, "Exception reloading roster", exception);
for (RosterLoadedListener listener : rosterLoadedListeners) { for (RosterLoadedListener listener : rosterLoadedListeners) {