localstack / localstack-java-utils

☕ Java utilities and JUnit integration for LocalStack
Apache License 2.0
75 stars 43 forks source link

Get 100 log lines #64

Closed AntonOellerer closed 3 years ago

AntonOellerer commented 3 years ago

The fetched number of log lines (10) was too low s.t. when a container took some time to boot the 404 messages from the request already took up too much space in the logs, and the Ready. token could not be detected. This commit increases the number of log lines to 100, so that this error should be alleviated, although it is of course not an optimal solution.

It might make sense too simply fetch all logs (with a -1 parameter to --tail), and let the user overwrite this value.

Closes #63

whummer commented 3 years ago

LGTM, thanks for fixing this @AntonOellerer ! 👍 We'll release a new version to Maven Central shortly.. Thanks