firebase-admin-dotnet icon indicating copy to clipboard operation
firebase-admin-dotnet copied to clipboard

Implementation of Cloud Storage function.

Open kiliczsh opened this issue 4 years ago • 1 comments

Original PR: https://github.com/firebase/firebase-admin-dotnet/pull/72 #71 Configuration keys are missing on the actions machines so that had to handle exceptions. Google was not allowing me to submit other users commits so that had to copy all, sorry @takuya1981. @googlebot I consent.

All credits to @takuya1981 .

kiliczsh avatar Feb 24 '21 17:02 kiliczsh

Thanks @kiliczsh for the PR (complete with integration tests). Storage support in .NET has always been a bit of a conundrum. In all other languages of the Admin SDK, we expose the "bucket" as the primary abstraction. For example in Node.js:

const admin = require('firebase-admin');
admin.initializeApp({
  storageBucket: 'example.bucket.name',
});

const bucket = admin.storage().bucket();

But as you already know, in the .NET Storage client, that abstraction does not exist. This results in a very different developer experience. I'd like to run this by our internal API review team, and see what they think about it. Since we have no way of passing a bucket name to the Storage client via Firebase app options, I'm not certain if this integration adds a lot of value.

hiranya911 avatar Feb 24 '21 19:02 hiranya911