Remove deadcode for pre-authorization

influxdata / influxdb

Scalable datastore for metrics, events, and real-time analytics

https://influxdata.com

Apache License 2.0

28.62k stars 3.54k forks source link

Remove deadcode for pre-authorization #15389

Open affo opened 4 years ago

affo commented 4 years ago

Some code for the pre-authorization is dead.

We should at least remove BucketsAccessed: https://github.com/influxdata/influxdb/blob/2a6cfe3c78f49100d3c17d266107b7efa99f1067/query/spec.go#L32

And the WalkIR function in Flux: https://github.com/influxdata/flux/blob/780ffd77ed641d0005fbc704f56c4e329b33cbf3/lang/compiler.go#L452

DOD:

understand how much of the pre-auth code can be removed
remove it
remove WalkIR from Flux

affo commented 4 years ago

@aanthony1243 as far as you know, it the whole pre-auth code or only BucketsAccessed to be removed?

Thank you

stale[bot] commented 4 years ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

affo commented 4 years ago

Was this fixed somehow?