Skip to content

Commit 9177fb9

Browse files
committed
fix wrong logging filter format, which causes 502 error
1 parent 97b0629 commit 9177fb9

1 file changed

Lines changed: 8 additions & 4 deletions

File tree

google-cloud-logging/src/test/java/com/google/cloud/logging/BaseSystemTest.java

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -468,12 +468,14 @@ public void testLoggingHandler() throws InterruptedException {
468468
logger.addHandler(handler);
469469
logger.setLevel(Level.INFO);
470470
logger.info("Message");
471+
String filter =
472+
"logName = projects/" + logging().getOptions().getProjectId() + "/logs/" + logName;
471473
Iterator<LogEntry> iterator =
472-
logging().listLogEntries(EntryListOption.filter("logName:" + logName)).iterateAll().iterator();
474+
logging().listLogEntries(EntryListOption.filter(filter)).iterateAll().iterator();
473475
while (!iterator.hasNext()) {
474476
Thread.sleep(500L);
475477
iterator =
476-
logging().listLogEntries(EntryListOption.filter("logName:" + logName)).iterateAll().iterator();
478+
logging().listLogEntries(EntryListOption.filter(filter)).iterateAll().iterator();
477479
}
478480
assertTrue(iterator.hasNext());
479481
LogEntry entry = iterator.next();
@@ -510,12 +512,14 @@ public void testSyncLoggingHandler() throws InterruptedException {
510512
logger.addHandler(handler);
511513
logger.setLevel(Level.WARNING);
512514
logger.warning("Message");
515+
String filter =
516+
"logName = projects/" + logging().getOptions().getProjectId() + "/logs/" + logName;
513517
Iterator<LogEntry> iterator =
514-
logging().listLogEntries(EntryListOption.filter("logName:" + logName)).iterateAll().iterator();
518+
logging().listLogEntries(EntryListOption.filter(filter)).iterateAll().iterator();
515519
while (!iterator.hasNext()) {
516520
Thread.sleep(500L);
517521
iterator =
518-
logging().listLogEntries(EntryListOption.filter("logName:" + logName)).iterateAll().iterator();
522+
logging().listLogEntries(EntryListOption.filter(filter)).iterateAll().iterator();
519523
}
520524
assertTrue(iterator.hasNext());
521525
LogEntry entry = iterator.next();

0 commit comments

Comments
 (0)