• David Benjamin's avatar
    Switch WebPushSender to credentials_mode. · 86aa928c
    David Benjamin authored
    This CL is part of a larger project to remove the LOAD_DO_NOT_* privacy
    load flags and replace them with the allow_credentials setting. See
    https://crbug.com/799935 and
    https://docs.google.com/document/d/1ntn9N7Ce2jozvvpWI0XbzJ7lJdwUjJXK07wp7rxrIN4
    for the motivation for this change.
    
    Using credential_mode makes it harder to forget
    LOAD_DO_NOT_SEND_AUTH_DATA along with the two cookie flags. This has two
    implications:
    
    - The request will no longer participate in HTTP auth if the server requests
      it. HTTP auth credentials identify the user, so private requests should
      not send them. Note this only affects server auth, not proxy auth. Unless
      the server your feature speaks to requests HTTP auth, this is a no-op.
    
    - The request will be pooled with sockets used for uncredentialed requests,
      rather than credentialed requests. This is not expected to meaningfully
      change behavior.
    
    If your code requires HTTP authentication, let me know. You may need to
    allow credentials for your request.
    
    Bug: 799935
    Change-Id: I64c4c87a16cfb70e1ffbb5abdcc9097c48ef9956
    Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1752745
    Auto-Submit: David Benjamin <davidben@chromium.org>
    Reviewed-by: default avatarJian Li <jianli@chromium.org>
    Commit-Queue: David Benjamin <davidben@chromium.org>
    Cr-Commit-Position: refs/heads/master@{#686598}
    86aa928c
web_push_sender.cc 8.19 KB