zydronium/google-nomulus
1
0
Fork 0
mirror of https://github.com/google/nomulus.git synced 2025-04-30 03:57:51 +02:00
Code Issues Projects Releases Packages Wiki Activity

Clean up RDAP action logs and request plain text (#213)

Browse source 
* Remove unused log argument

* Use the right accept-encoding

By default we request gzip and theoretically we'd decode it
automatically on our end but for some reason that's not working. I
tested this on Alpha and it worked
This commit is contained in:
gbrodman 2019-08-06 14:00:09 -04:00 • committed by GitHub
parent 8240dee691
commit 8d289dbb54
1 changed files with 2 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
core/src/main/java/google/registry/rdap/UpdateRegistrarRdapBaseUrlsAction.java
View file

@ -130,6 +130,7 @@ public final class UpdateRegistrarRdapBaseUrlsAction implements Runnable {
try { try {
HttpRequest request = HttpRequest request =
requestFactory.buildGetRequest(new GenericUrl(String.format(LIST_URL, tld))); requestFactory.buildGetRequest(new GenericUrl(String.format(LIST_URL, tld)));
request.getHeaders().setAcceptEncoding("identity");
request.getHeaders().setCookie(String.format("%s=%s", COOKIE_ID, id)); request.getHeaders().setCookie(String.format("%s=%s", COOKIE_ID, id));
HttpResponse response = request.execute(); HttpResponse response = request.execute();
@ -175,7 +176,7 @@ public final class UpdateRegistrarRdapBaseUrlsAction implements Runnable {
} catch (Throwable e) { } catch (Throwable e) {
// Login failures are bad but not unexpected for certain TLDs. We shouldn't store those // Login failures are bad but not unexpected for certain TLDs. We shouldn't store those
// but rather should only store useful Throwables. // but rather should only store useful Throwables.
logger.atWarning().log("Error logging in to MoSAPI server: " + e.getMessage(), e); logger.atWarning().log("Error logging in to MoSAPI server: " + e.getMessage());
continue; continue;
} }
try { try {