Skip to content
This repository was archived by the owner on May 7, 2021. It is now read-only.

Conversation

@sastels
Copy link
Contributor

@sastels sastels commented Mar 13, 2020

Description

Add some extra console.logs to try to figure out what's pinging our app. Does the first 20 requests then stops printing out.

Required followup work

revert this PR when not needed anymore

Checklist:

  • I have looked at my code on GitHub and it all looks good (ex: no random commented out code or console.logs)
  • I have added and needed tests for my changes (in particular for new screens)
  • I have added a comment to any confusing code

@sastels sastels requested review from davidheerema and ngosset March 13, 2020 16:30
Copy link
Contributor

@davidheerema davidheerema left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks Steve.

@sastels sastels merged commit b95d8d6 into release-rcmp-prod Mar 13, 2020
@sastels sastels deleted the enable-debug-logging branch March 13, 2020 16:57
davidheerema added a commit that referenced this pull request Mar 27, 2020
* FIX CORS for HTTPS (#1665)

Yesterday CAFC moved to HTTPS, this broke our allowed origins.

* Emergency Fix - Don't count non-CAFC referrals (#1672)

* Only count referrers from CAFC

* Fixed typo

* Referrer urls are less predictable than origin? Just look at the host portion

* temporarily enable request debugging (#1673)

Co-authored-by: ngosset <[email protected]>
Co-authored-by: dhcdn <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants