irods / python-irodsclient

A Python API for iRODS
Other
62 stars 72 forks source link

[550] Add support for client hints #551

Closed stsnel closed 2 months ago

stsnel commented 2 months ago

...I do have one question, though. Is the irods.session the right place for this? I don't use PRC enough to know of a better place, but just want to make sure we considered other designs (a new Model, a property of a Connection, etc.).

Right, I chose session.py initially, since it already has the server_version attribute, and it seemed logical to me to put attributes that show information about the server in the same place. But I'm also happy to move it elsewhere if that fits the design better.

trel commented 2 months ago

Note - this is information from the connected server only.... right?

alanking commented 2 months ago

Sold. Merging. Thanks!