mozilla / fxa-customs-server

95%

master: 95%

LAST BUILD ON BRANCH phil/issue-205

branch: phil/issue-205

CHANGE BRANCH

x

Reset

• phil/issue-205
• i12
• master
• add-blocklist-module
• dev-deps-0720
• issue-120-remove-account-lockout
• v0.66.0
• remove-lockout-leftovers
• backport
• v0.67.0
• add-optional-newrelic
• v0.69.0
• v0.68.0
• unblock
• rate-limit-verify-code
• fix-reload-of-nested-settings
• up-restify
• v0.71.0
• unblockvs-ip-blocklist
• backport-suspect-requests
• no-allowed-ips-by-default
• dont-log-msg
• update-blocklist-level1
• fix-fstat-mtime
• v0.72.0
• fix-per-ip-rate-limiting
• shrinkwrap-npm
• config-set-merge
• issue-154-update-contrib-docs
• rm-mockmyid
• v0.78.0
• round-ban-periods-up
• v0.79.0
• add-sms-support
• add-debug-support
• issue-165-override-blocklist-blocking
• throw-on-enoent
• unref-all-timers
• no-merge-config-lists
• i164
• dockerize
• v0.80.0
• reputation-service-moar-logging
• v1.81.0
• fix-process-shutdown
• v1.82.0
• add-sms-docs
• sms-more-testing
• v1.83.0
• issue-190-fix-sms-rate-limit
• v1.84.0
• docker-node-4.8.2
• v1.85.0
• extract-update-blocks
• issue-188-add-sms-block-per-email
• issue-180-add-additional-email-rate-limit
• pushonempty-rejection
• v1.86.0
• custom-docker-br
• phil/issue-201
• v1.88.0
• phil/account-access-fail
• phil/ffs-failing-test
• phil/issue-205-alt
• phil/issue-205-baby-steps
• node6
• v1.89.0
• v1.90.0
• i204
• node-sec
• v1.91.0
• phil/document-memcached
• v1.92.0
• v1.98.0
• travis-node-8
• changelog
• fix-newrelic-must-be-first
• dockerpush-fix-newrelic-must-be-first
• dockerpush-fix-newrelic-native-needs-build-tools
• v1.100.0
• issue-217-allow-test-phone-numbers
• fixes-for-node-8
• v1.101.0
• v1.102.0
• fix-use-node-6.12.3
• sentry
• issue-222
• v1.103.0
• nsp-updates
• v1.106.0
• v1.108.0
• pb/bug-1447452
• node-6.14.0
• add-totp-rate-limit
• v1.109.0
• update-tests
• bug-1448854-sms-rate-limit-interval
• private-tag-docs
• v1.110.0
• train-110
• n8
• train-111
• update-newrelic-version
• shrink-latest
• v1.112.0

Build # 709

Build Type

push

travis-ci

Committed by philbooth

Commit Message

(iprecord): ignore prior rate-limiting for account access actions

Requests to /check with allow-listed email addresses cause the IP record
to to be marked as rate-limited, even though those requests aren't
actually blocked. Any subsequent requests to /checkIpOnly will fetch the
rate-limited IP record from memcached and block the request incorrectly.
This is a problem for the content server functional tests.

The change here ignores any previous rate-limiting for account access
actions, effectively putting them into a separate rate-limiting bucket.

Run Details

397 of 435 branches covered (91.26%)

1 of 1 new or added line in 1 file covered. (100.0%)

1026 of 1077 relevant lines covered (95.26%)

5546.28 hits per line