search

run-llama / llama_index

LlamaIndex is a data framework for your LLM applications
https://docs.llamaindex.ai
MIT License
36.74k stars 5.27k forks source link

feat: add get_permission_info to resource reader mixin + readers #16835

Closed EmanuelCampos closed 1 day ago

logan-markewich commented 1 week ago

@EmanuelCampos friendly ping on the failing tests here

Javtor commented 2 days ago

What's the reasoning behind adding this method? If we want to decouple permissions info from general resource info, should we still keep the logic for adding permissions info in get_resource_info?

EmanuelCampos commented 2 days ago

What's the reasoning behind adding this method? If we want to decouple permissions info from general resource info, should we still keep the logic for adding permissions info in get_resource_info?

At first moment keeping it for backwards compatibility, but yes the goal is decoupling permissions from general metadata