API for accessing ETag values about firebase-admin-python HOT 4 CLOSED

@caseycrogers, you said in the comments for #54 that

As for your interest in adding low-level access to ETags-this seems reasonable to me. However, there is a caveat:

As-is, the ETag of a given value at one path in the database will be the same as the ETag of the same value at any other path.

Additionally, the ETag of values will not change with time. However, we do not document these two properties and do not guarantee that EITHER of them will remain true in the future. As such, we strongly discourage developers from fetching ETag values and then saving them to be used with later transactions at the same or different locations.

I don't think that this is a problem in the case of caching. If the data is the same, and hence the ETag is the same, then the cache is not stale. Also, it does not matter here if the ETag does not change as a function of time (I would certainly think that we would not want it to!)

from firebase-admin-python.

By change as a function time I mean that in the future we may choose to change our algorithm for producing ETags. We have no such plans at the moment but want to keep the option open for ourselves.

For caching it should be fine. Another note that you might find very helpful-while implementing the conditional requests logic I also implemented full support for the if-none-match header. However, since this feature was well out of scope of the project (allowing transactions in REST) we decided to keep the functionality but not document it. If you're doing any caching related stuff I highly recommend you use if-none-match.

It may be even worthwhile to add under-the-hood caching functionality with if-none-match to this library's default GET implementation. Preferably with a flag to turn it on or off.

Requests for non-stale data in the cache would incur one round trip (to verify the cached value) and minimal bandwidth. Stale data would require two round trips and slightly higher bandwidth than just a single standard GET.

from firebase-admin-python.

Just submitted a pull request for this. I think I'll hold off on the under-the-hood caching for now, as that seems like it'll be pretty complicated.

from firebase-admin-python.

I will resolve this, since get(etag=True) and set_if_unchanged() are already implemented and released. I also just sent out #73 to add the get_if_changed() method. Thanks @alexanderwhatley for all your hard work on getting this implemented, and thanks @caseycrogers for all the help.

from firebase-admin-python.