forked from Expensify/App
-
Notifications
You must be signed in to change notification settings - Fork 0
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Merge branch 'Expensify:main' into @kosmydel/handle-invisible-characters
- Loading branch information
Showing
566 changed files
with
10,108 additions
and
11,349 deletions.
There are no files selected for viewing
Validating CODEOWNERS rules …
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,2 +1,5 @@ | ||
# Every PR gets a review from an internal Expensify engineer | ||
* @Expensify/pullerbear | ||
|
||
# Every PR that touches redirects gets reviewed by ring0 | ||
docs/redirects.csv @Expensify/infra |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,125 @@ | ||
#!/bin/bash | ||
# | ||
# Adds new routes to the Cloudflare Bulk Redirects list for communityDot to helpDot | ||
# pages. Does some basic sanity checking. | ||
|
||
set -e | ||
|
||
source scripts/shellUtils.sh | ||
|
||
info "Adding any new redirects from communityDot to helpDot" | ||
|
||
declare -r LIST_ID="20eb13215038446a98fd69ccf6d1026d" | ||
declare -r ZONE_ID="$CLOUDFLARE_ACCOUNT_ID" | ||
declare -r REDIRECTS_FILE="docs/redirects.csv" | ||
|
||
function checkCloudflareResult { | ||
RESULTS=$1 | ||
RESULT_MESSAGE=$(echo "$RESULTS" | jq .success) | ||
|
||
if ! [[ "$RESULT_MESSAGE" == "true" ]]; then | ||
ERROR_MESSAGE=$(echo "$RESULTS" | jq .errors) | ||
error "Error calling Cloudfalre API: $ERROR_MESSAGE" | ||
exit 1 | ||
fi | ||
} | ||
|
||
declare -a ITEMS_TO_ADD | ||
|
||
while read -r line; do | ||
# Split each line of the file into a source and destination so we can sanity check | ||
# and compare against the current list. | ||
read -r -a LINE_PARTS < <(echo "$line" | tr ',' ' ') | ||
SOURCE_URL=${LINE_PARTS[0]} | ||
DEST_URL=${LINE_PARTS[1]} | ||
|
||
# Make sure the format of the line is as execpted. | ||
if [[ "${#LINE_PARTS[@]}" -gt 2 ]]; then | ||
error "Found a line with more than one comma: $line" | ||
exit 1 | ||
fi | ||
|
||
# Basic sanity checking to make sure that the source and destination are in expected | ||
# subdomains. | ||
if ! [[ $SOURCE_URL =~ ^https://community\.expensify\.com ]]; then | ||
error "Found source URL that is not a community URL: $SOURCE_URL" | ||
exit 1 | ||
fi | ||
|
||
if ! [[ $DEST_URL =~ ^https://help\.expensify\.com ]]; then | ||
error "Found destination URL that is not a help URL: $DEST_URL" | ||
exit 1 | ||
fi | ||
|
||
info "Source: $SOURCE_URL and destination: $DEST_URL appear to be formatted correctly." | ||
|
||
ITEMS_TO_ADD+=("$line") | ||
|
||
# This line skips the first line in the csv because the first line is a header row. | ||
done <<< "$(tail +2 $REDIRECTS_FILE)" | ||
|
||
# Sanity check that we should actually be running this and we aren't about to delete | ||
# every single redirect. | ||
if [[ "${#ITEMS_TO_ADD[@]}" -lt 1 ]]; then | ||
error "No items found to add, why are we running?" | ||
exit 1 | ||
fi | ||
|
||
# This block builds a single JSON object with all of our updates so we can | ||
# reduce the number of API calls we make. You cannot add any logging or anything | ||
# that prints to std out to this block or it will break. We capture all of the std out | ||
# from this loop and pass it to jq to build the json object. Any non-json will break the | ||
# jq call at the end. | ||
PUT_JSON=$(for new in "${ITEMS_TO_ADD[@]}"; do | ||
read -r -a LINE_PARTS < <(echo "$new" | tr ',' ' ') | ||
SOURCE_URL=${LINE_PARTS[0]} | ||
DEST_URL=${LINE_PARTS[1]} | ||
# We strip the prefix here so that the rule will match both http and https. Since vanilla will eventially be removed, | ||
# we need to catch both because we will not have the http > https redirect done by vanilla anymore. | ||
NO_PREFIX_SOURCE_URL=${SOURCE_URL/https:\/\//} | ||
jq -n --arg source "$NO_PREFIX_SOURCE_URL" --arg dest "$DEST_URL" '{"redirect": {source_url: $source, target_url: $dest}}' | ||
done | jq -n '. |= [inputs]') | ||
|
||
info "Adding redirects for $PUT_JSON" | ||
|
||
# We use PUT here instead of POST so that we replace the entire list in place. This has many benefits: | ||
# 1. We don't have to check if items are already in the list, allowing this script to run faster | ||
# 2. We can support deleting redirects this way by simply removing them from the list | ||
# 3. We can support updating redirects this way, in the case of typos or moved destinations. | ||
# | ||
# Additionally this API call is async, so after we finish it, we must poll to wait for it to finish to | ||
# to know that it was actually completed. | ||
PUT_RESULT=$(curl -s --request PUT --url "https://api.cloudflare.com/client/v4/accounts/$ZONE_ID/rules/lists/$LIST_ID/items" \ | ||
--header 'Content-Type: application/json' \ | ||
--header "Authorization: Bearer $CLOUDFLARE_LIST_TOKEN" \ | ||
--data "$PUT_JSON") | ||
|
||
checkCloudflareResult "$PUT_RESULT" | ||
OPERATION_ID=$(echo "$PUT_RESULT" | jq -r .result.operation_id) | ||
|
||
DONE=false | ||
|
||
# Poll for completition | ||
while [[ $DONE == false ]]; do | ||
CHECK_RESULT=$(curl -s --request GET --url "https://api.cloudflare.com/client/v4/accounts/$ZONE_ID/rules/lists/bulk_operations/$OPERATION_ID" \ | ||
--header 'Content-Type: application/json' \ | ||
--header "Authorization: Bearer $CLOUDFLARE_LIST_TOKEN") | ||
checkCloudflareResult "$CHECK_RESULT" | ||
|
||
STATUS=$(echo "$CHECK_RESULT" | jq -r .result.status) | ||
|
||
# Exit on completed or failed, other options are pending or running, in both cases | ||
# we want to keep polling. | ||
if [[ $STATUS == "completed" ]]; then | ||
DONE=true | ||
fi | ||
|
||
if [[ $STATUS == "failed" ]]; then | ||
ERROR_MESSAGE=$(echo "$CHECK_RESULT" | jq .result.error) | ||
error "List update failed with error: $ERROR_MESSAGE" | ||
exit 1 | ||
fi | ||
done | ||
|
||
success "Updated lists successfully" |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.