Skip to content
Snippets Groups Projects
Commit 3281b70d authored by Marius Dumitru Florea's avatar Marius Dumitru Florea
Browse files

XWIKI-19913: PDF export sometimes fails when setting the cookies for the headless Chrome browser

* Cookies can be duplicated. The code shouldn't break when it happens. Also, it's good to log all cookies, even duplicated ones.
parent f575bf0d
No related branches found
No related tags found
No related merge requests found
...@@ -307,8 +307,8 @@ public CookieParam toCookieParam(Cookie servletCookie) ...@@ -307,8 +307,8 @@ public CookieParam toCookieParam(Cookie servletCookie)
*/ */
public void setCookies(ChromeDevToolsService tabDevToolsService, List<CookieParam> cookies) public void setCookies(ChromeDevToolsService tabDevToolsService, List<CookieParam> cookies)
{ {
this.logger.debug("Setting cookies [{}].", this.logger.debug("Setting cookies [{}].", cookies.stream()
cookies.stream().collect(Collectors.toMap(CookieParam::getName, CookieParam::getValue))); .map(cookie -> String.format("%s: %s", cookie.getName(), cookie.getValue())).collect(Collectors.toList()));
Network network = tabDevToolsService.getNetwork(); Network network = tabDevToolsService.getNetwork();
network.enable(); network.enable();
network.clearBrowserCookies(); network.clearBrowserCookies();
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment