Hacktoberfest 2026: the issues maintainers tagged for October, open and beginner-friendly. Browse Hacktoberfest issues

deleteUsers(uids: string[]) Not Working

Open
#2,419 1 comment 0 reactions 1 assignee View on GitHub

@pragatimodi is already working on this.

Since Jan 10, 2024.

Assessment

This issue has not been assessed yet.

Description

api: auth

deleteUsers is only deleting the first entry of the provided array of uid, see the following:

  • Relevant code snippet:

    --- SNIP ---
  if (httpMethod === "DELETE") {
        const { id } = event.queryStringParameters;
        if (!id) return handleError(new Error("Please provide an id."), 405);

        const uids = id.split(",");
        return handleDeleteRequest(tenantAuth, uids);
      }
--- SNIP ---

const handleDeleteRequest = async (tenantAuth, uids) => {
  try {
    console.log("handleDeleteRequest -> uids:", uids);
    const deleteUserResults = await tenantAuth.deleteUsers(uids);

    return {
      body: JSON.stringify({ results: deleteUserResults }),
      headers: { "Content-Type": "application/json" },
      statusCode: 200,
    };
  } catch (error) {
    return handleError(error);
  }
};

  • uids value:

    handleDeleteRequest -> uids: [ 'cbHxLH4TtKN8tREDACTED', ' PLrrHeqlphcHP1REDACTED' ]
Response with status 200 in 855 ms.

  • Server's response:

    {"results":{"failureCount":0,"successCount":2,"errors":[]}}

Although success count shows 2 when going to the console only the first user is deleted. It seems that this function is buggy...

Dominant language
TypeScript
Stars
1.7k
Forks
419
Avg merge
4d 20h
Merged PRs (30d)
16

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from firebase/firebase-admin-node

All issues in firebase/firebase-admin-node

Similar issues

More TypeScript issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.