Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Fix setting a user's external_id via the admin API returns 500 and deletes users existing external mappings if that external ID is already mapped #11051

Merged
merged 7 commits into from
Oct 21, 2021
Merged

Fix setting a user's external_id via the admin API returns 500 and deletes users existing external mappings if that external ID is already mapped #11051

merged 7 commits into from
Oct 21, 2021

Commits on Oct 12, 2021

  1. Fix setting a user's external_id via the admin API returns 

    500 and deletes users existing external mappings if that external ID is
already mapped
    @dklimpel
    dklimpel committed Oct 12, 2021
    Configuration menu
    Copy the full SHA
    49cc814 View commit details
    Browse the repository at this point in the history

  2. newsfile

    @dklimpel
    dklimpel committed Oct 12, 2021
    Configuration menu
    Copy the full SHA
    7fd4c49 View commit details
    Browse the repository at this point in the history

Commits on Oct 13, 2021

  1. add own exception

    @dklimpel
    dklimpel committed Oct 13, 2021
    Configuration menu
    Copy the full SHA
    caedb57 View commit details
    Browse the repository at this point in the history

  2. rewrite db calls 

    - rename `record_and_remove_user_external_id` to
`replace_user_external_id`
- modify `replace_user_external_id` to delete all mappings and add given
mapping
- restore `remove_user_external_id` to orign, no `_txn` call is needed
- add `remove_user_external_ids` to delete all mappings with one sql
query
- also add `_remove_user_external_ids_txn`
- change rest handler to use new functions
    @dklimpel
    dklimpel committed Oct 13, 2021
    Configuration menu
    Copy the full SHA
    949b1ae View commit details
    Browse the repository at this point in the history

  3. add exception handling to record_user_external_id and 

    create user in admin API
    @dklimpel
    dklimpel committed Oct 13, 2021
    Configuration menu
    Copy the full SHA
    f6a2092 View commit details
    Browse the repository at this point in the history

Commits on Oct 14, 2021

  1. move functions

    @dklimpel
    dklimpel committed Oct 14, 2021
    Configuration menu
    Copy the full SHA
    e40bcd4 View commit details
    Browse the repository at this point in the history

  2. mypy

    @dklimpel
    dklimpel committed Oct 14, 2021
    Configuration menu
    Copy the full SHA
    fa32735 View commit details
    Browse the repository at this point in the history