GuVAnj8Gv3RJ/NeosAccountDownloader

`*.Storage.json` Files Only Have `null` Content

Closed this issue · 5 comments

Any *.Storage.json files created for Groups and Members of Groups all contain null in them. What is the purpose of these files? Should they contain the number of bytes free and used for a group and for a member?

image

Additionally, what is with the following endpoints?

api/users/(userId)/storage
api/groups/(groupId)/storage
api/groups/(groupId)/members/(userId)/storage

These endpoints are being fetched with the GET verb from methods GetStorage and GetMemberStorage in CloudXInterfaceExtensions.cs. Those methods are called when the downloader is gathering the storage information for the group and its members, but they are returning null back. Testing these endpoints with the CloudX client is also producing 404.

image

These endpoints are nowhere to be found in CloudX.Shared, and the models Storage, QuotaShare, QuotaSource, and QuotaType in Storage.cs are used to map to the returned JSON from these endpoints.

These files were requested by a user and sometimes appear in certain situations. We can update the code to not write them when they are not present though.

Thanks!

They all contain null. The best way to store storage information would be to use the other endpoints. They are exposed with a method from CloudXInterface, but I cannot remember which one. Here are the endpoints that might be better to use:

image

The data stored is different and sometimes not null. I will update the code to only store when It is not null.

Thank you.

But that does not make sense how it would sometimes not be null. Those endpoints are not available in CloudX.Shared, so no one on Neos can interact with those endpoints:

api/users/(userId)/storage
api/groups/(groupId)/storage
api/groups/(groupId)/members/(userId)/storage

Here are images of debugging this particular area:

image image

The only situation that these endpoints could be reached is if this tool is being used elsewhere internally where these endpoints are understood. Anyway, I understand these are the requirements you must fulfill whether or not they make sense.

Fixed in: a85b727