Skip to content

LG-7431 Logged In User Change Password#6948

Merged
ThatSpaceGuy merged 2 commits intomainfrom
LG-7431-Change-Password
Sep 13, 2022
Merged

LG-7431 Logged In User Change Password#6948
ThatSpaceGuy merged 2 commits intomainfrom
LG-7431-Change-Password

Conversation

@ThatSpaceGuy
Copy link
Contributor

changelog: Internal, Attempts API, Track additional events

changelog: Internal, Attempts API, Track additional events
@ThatSpaceGuy ThatSpaceGuy requested a review from a team September 13, 2022 15:31
Copy link
Contributor

@zachmargolis zachmargolis left a comment

Choose a reason for hiding this comment

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

LGTM

@ThatSpaceGuy ThatSpaceGuy merged commit 69d7b7c into main Sep 13, 2022
@ThatSpaceGuy ThatSpaceGuy deleted the LG-7431-Change-Password branch September 13, 2022 16:35
@aduth aduth mentioned this pull request Sep 15, 2022
aduth added a commit that referenced this pull request Sep 15, 2022
* LG-6497: Revert Memorable Date Changes (#6940)

* Revert "LG-6497: Minor style/doc fixes related to memorable date change (#6929)"

This reverts commit e0e8ad4.

* Revert "LG-6497: Create and use new Memorable Date component in State ID step of IPP flow (#6713)"

This reverts commit cead2c1.

* [skip changelog]

* Build frontend analytics method parameters from method signature (#6927)

* Build frontend analytics method parameters from method signature

**Why**:

- To avoid confusion where a developer would expect analytics method parameters to be fulfilled from the frontend payload (#6791)
- To ensure that all of the analytics methods are explicit and documented in the parameters they're expecting

See: #6918 (comment)

changelog: Internal, Analytics, Improve frontend analytics payload documentation

* Fix YARDoc parameter name documentation

* Restore flow path parameter to front-end mapped events

* Add flow_path handling to specs

* Destructure, use compact for kwarg aggregation

* Extract hash_from_method_kwargs to service class

See: #6927 (comment)

* Extract FrontendLogger service class from FrontendLogController

See: #6927 (comment)

* Remove leftover code from earlier iterations

* Collapse MethodSignatureHashBuilder to FrontendLogger

See: #6927 (comment)

* LG-6497: Reintroduce Memorable Date on IPP State ID Page (#6943)

* Revert "LG-6497: Revert Memorable Date Changes (#6940)"

This reverts commit d2500a8.

* [skip changelog]

* Try to fix intermittent failure for in-person feature spec (#6933)

**Why**: So that our builds pass reliably.

changelog: Internal, Automated Testing, Reduce intermittent failures in automated testing

* LG-7431 Logged In User Change Password (#6948)

* LG-7431 Logged In User Change Password

changelog: Internal, Attempts API, Track additional events

* Qualify script rails executable properly with bin/ (#6951)

changelog: Internal, Upcoming Features, Qualify script rails executable properly with bin/

* LG-6497: Workaround New Relic addEventListener bug and improve error styles (#6938)

* LG-6497: Circumvent New Relic event bug

* LG-6497: Ensure memorable date correctly assigns errors for error styling

* [skip changelog]

* Update app/javascript/packages/memorable-date/index.spec.ts

Co-authored-by: Sheldon Bachstein <sheldon.bachstein@gsa.gov>

* Revert "LG-6497: Revert Memorable Date Changes (#6940)"

This reverts commit d2500a8.

* [skip changelog]

* Update app/javascript/packages/memorable-date/index.ts

Co-authored-by: Andrew Duthie <andrew.duthie@gsa.gov>

Co-authored-by: Sheldon Bachstein <sheldon.bachstein@gsa.gov>
Co-authored-by: Andrew Duthie <andrew.duthie@gsa.gov>

* Refactor PhoneFinder (#6920)

This commit refactors the PhoneFinder proofer client so that it does not user the `Proofer::Base` super class any more. It converts the PhoneFinder proofer into a plain old ruby object with its own result object.

[skip changelog]

Co-authored-by: Jonathan Hooper <jonathan.hooper@gsa.gov>

* ThreatMetrix JS verification (LG-7518) (#6937)

* Add ThreatMetrix code signing cert to config

Cert includes public key used to verify signature on ThreatMetrix Javascript

* Add job to verify ThreatMetrix javascript

Periodically request TMX JS and validate the signature at the end of the file

* Remove stale job stuff

* Don't hardcode test certs

Generate keys / certs so we don't have scary-looking private keys in repo

* Minor tweak to session_id arg

changelog: Upcoming Features, ThreatMetrix, Periodically check signature on ThreatMetrix Javascript

* Use pack to do hex-to-binary conversion

* how do you spell Javascript

* Fail if signing certificate is expired

* Run ThreatMetrixJsVerificationJob on 1h intervals

* Remove cert from application.default.yml

will be in config, just not going to include in code

* Lint

* Trigger devops on merges to main. (#6944)

* Trigger devops on merges to main.

* typo

* testing

[skip changelog]

* Only run on main

* Update total-monthly-auths report to pull from the raw table (#6952)

**Why**: We've found some discrepancies with the aggregated monthly
table so this helps us have more precise reports

changelog: Internal, Reporting, Update billing reports to be more accurate

* Log digest of OIDC "code" param (LG-7440) (#6942)

* Log digest of OIDC "code" param (LG-7440)

**Why**: to assist with debugging requests from partners

* Work around nil code

**How**: By submitting the form a second time after the identity
has been linked, because we need the session uuid from that linkage
for the code, for the param.

Somehow it was getting into the success_redirect_uri before

changelog: Internal, Logging, Log hash of OpenID Connect "code" param

Co-authored-by: Andrew Duthie <andrew.duthie@gsa.gov>

* update ready to verify presenter to use enrollment_established_at (#6950)

* update ready to verify presenter to use enrollment_established_at

* [skip changelog]

* refactor

* fix lint errors

* added new event for idv phone send link rate limit (#6958)

changelog: Internal, Attempts API, Track additional events

* Lg 7545 password change reauth (#6954)

* LG-7545 Password Change Reauth Submit

changelog: Internal, Attempts API, Track additional events

* LG-7411 Analytics event for TM Proofing Failure Page (#6953)

* added setup errors visited

* Add redirect tracking to contact url on sorry page

* add changelog

changelog: Internal, Analytics, Add analytics to sorry for proofing page

* add tracker method for contact page analytics

* LG-6497: Set min date for State ID page to January 1, 1900 (#6960)

* LG-6497: Set min date for State ID page to January 1, 1900; remove leading zero from day

* [skip changelog]

* LG-7203: Use DB for service provider IPP feature enablement (#6945)

* LG-7203: Add in_person_proofing_enabled field to service_providers; update feature check to use new field

* LG-7203: Replace use of hardcoded IPP issuer config and update tests

* changelog: Upcoming Features, In-person proofing, Use DB for configuring service providers

* LG-7203: Switch to leaner query query clear expected value for in_person_proofing_enabled

* LG-7547-Password-Change-Reauthentication-Rate-Limit (#6955)

* LG-7547-Password-Change-Reauthentication-Rate-Limit

changelog: Internal, Attempts API, Track additional events

* LG-7195 | Adds reproof completed event (#6957)

changelog: Internal, Attempts API, Adds reproofing complete event

* Update Password Change to Profile Change (#6962)

changelog: Internal, Attempts API, Track additional events

* Enforce YARD parameter documentation for tracker_events.rb, fix errors (#6964)

[skip changelog]

Co-authored-by: Tim Bradley <90272033+NavaTim@users.noreply.github.com>
Co-authored-by: Luis H. Matos <ThatSpaceGuy@users.noreply.github.com>
Co-authored-by: Gene M. Angelo, Jr <web.gma@gmail.com>
Co-authored-by: Sheldon Bachstein <sheldon.bachstein@gsa.gov>
Co-authored-by: Kimball Bighorse <kimball.bighorse@gsa.gov>
Co-authored-by: Jonathan Hooper <jonathan.hooper@gsa.gov>
Co-authored-by: Matt Hinz <matthinz@gmail.com>
Co-authored-by: Alex Kritikos <alex.kritikos@gsa.gov>
Co-authored-by: Zach Margolis <zachmargolis@users.noreply.github.com>
Co-authored-by: Shannon A <20867088+svalexander@users.noreply.github.com>
Co-authored-by: Rwolfe-Nava <87499456+Rwolfe-Nava@users.noreply.github.com>
Co-authored-by: Alex Bradley <alexander.bradley@gsa.gov>
Co-authored-by: Matt Wagner <mattwagner@navapbc.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants