Skip to content

A
apollo
Commit 8737d49c authored by Jason Song's avatar Jason Song
Browse files
Options

prevent crash when app.id is not set

parent a0bee9d4
Show whitespace changes
Inline Side-by-side
Showing with 16 additions and 9 deletions
apollo-client/src/main/java/com/ctrip/framework/apollo/internals/RemoteConfigRepository.java
View file @ 8737d49c
...@@ -259,6 +259,7 @@ public class RemoteConfigRepository extends AbstractConfigRepository { ...@@ -259,6 +259,7 @@ public class RemoteConfigRepository extends AbstractConfigRepository {
} }
private void scheduleLongPollingRefresh() { private void scheduleLongPollingRefresh() {
try {
final String appId = m_configUtil.getAppId(); final String appId = m_configUtil.getAppId();
final String cluster = m_configUtil.getCluster(); final String cluster = m_configUtil.getCluster();
final String dataCenter = m_configUtil.getDataCenter(); final String dataCenter = m_configUtil.getDataCenter();
...@@ -268,6 +269,12 @@ public class RemoteConfigRepository extends AbstractConfigRepository { ...@@ -268,6 +269,12 @@ public class RemoteConfigRepository extends AbstractConfigRepository {
doLongPollingRefresh(appId, cluster, dataCenter); doLongPollingRefresh(appId, cluster, dataCenter);
} }
}); });
} catch (Throwable ex) {
ApolloConfigException exception =
new ApolloConfigException("Schedule long polling refresh failed", ex);
Cat.logError(exception);
logger.warn(ExceptionUtil.getDetailMessage(exception));
}
} }
private void doLongPollingRefresh(String appId, String cluster, String dataCenter) { private void doLongPollingRefresh(String appId, String cluster, String dataCenter) {
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment