Mike Eltsufin
Mike Eltsufin
Reopening, because we need the fix in master as well due to a customer issue.
Can we replace the current config with something that rolls request logs more aggressively? Either keep only several hours worth of logs, or restrict by size? ``` XML /request.yyyy_mm_dd.log 2...
It sounds like there is no minimally invasive change that we can do right now, except for changing retainDays to 1.
+1 for removing the Jetty request log altogether for now.
@wangzilong Please try replacing `vm: True` with `env: flex`. Also, unless Java 7 is a requirement for you, please change `runtime: java7` to `runtime: java`.
@ludoch Do you think we should merge this change in and let users try it out, or wait for Bin?
@gregw Agreed. This kind of code is always error prone and should be carefully reviewed. I haven't had a chance to look at it in detail since being away this...
@diegomarquezp I'm a little late to this, but is there a related issue I can review that this is trying to solve?
@TimurSadykov Is this something you're familiar with? I see that you modified `GoogleIdTokenVerifier` recently.