Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix: add customAttributes to edit account request #13

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

schultek
Copy link

The customAttributes field is not encoded in the final request object, therefore when calling setCustomUserClaims the claims are never updated.

@spoeck
Copy link

spoeck commented Mar 10, 2021

I have tested this PR, works as expected -> should be merged

schultek and others added 3 commits July 12, 2021 16:26
# Conflicts:
#	lib/src/admin.dart
#	lib/src/app.dart
#	lib/src/app/app.dart
#	lib/src/app/app_extension.dart
#	lib/src/auth.dart
#	lib/src/auth/auth_api_request.dart
#	lib/src/auth/credential.dart
#	lib/src/auth/token_verifier.dart
#	lib/src/auth/user_record.dart
#	lib/src/credential.dart
#	lib/src/database.dart
#	lib/src/storage.dart
#	lib/src/utils/error.dart
#	lib/src/utils/validator.dart
#	pubspec.yaml
#	test/app_test.dart
#	test/auth_test.dart
#	test/firebase_admin_test.dart
#	test/resources/mocks.dart
#	test/storage_test.dart
@sgehrman
Copy link

sgehrman commented Dec 5, 2021

Just spent hours trying to figure out why this isn't working.

Any reason why this hasn't been merged yet?

@Cobolt77
Copy link

Please merge with master :)

@sgehrman
Copy link

sgehrman commented Aug 3, 2022

What is going on?

@livtanong
Copy link

I think maybe this hasn't been merged because there's a lot of other changes in this PR. And there's this entire other method added as well: https://github.com/appsup-dart/firebase_admin/pull/13/files#diff-0b1edb4d91ddbe43e7a03462af85a8b215c06e6a3b988f8fd2a17f7b7e27690aR38

So perhaps to aid in this being merged in, this PR could be split up into three parts:

  1. The actual fix for customAttributes (Seems like this is just one line)
  2. The new method as a proposal
  3. All other style and minor refactorings

@schultek
Copy link
Author

I think this just isn't maintained anymore. There was no activity by the owner for a long time on any issue or pr.

I have forked this for my own use a while ago, you can use this if you want, but I'm not actively working on it: https://github.com/schultek/firebase_admin

@sgehrman
Copy link

Two years later and still broken.

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.

5 participants