Admin facade missing from schema

Bug #1885324 reported by Jeff Pihach
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Canonical Juju
Fix Released
High
Simon Richardson

Bug Description

Revision history for this message
Heather Lanigan (hmlanigan) wrote :

Investigation shows they were never included. The schema generation didn't start at the apiserver directory, but lower.

Changed in juju:
status: New → Triaged
importance: Undecided → High
Revision history for this message
Simon Richardson (simonrichardson) wrote :

It's currently not exposed[1] via the AllFacades, I question if it should be exposed from a security perspective (although it's only through obscurity that you can't access it)?

I think we should ask if we do expose it, why it's left off from the list of AllFacades. I'm unsure what the original intent was[2] and the commit message says nothing about the why?

1. https://github.com/juju/juju/blob/develop/apiserver/allfacades.go#L381-L400
2. https://github.com/juju/juju/commit/2183fbd2752baed4113f9eb690bdd0ed8b195daa

Revision history for this message
Jeff Pihach (hatch) wrote :

You also cannot use the API without the ability to log into it so it is kind of a requirement :D

Revision history for this message
Simon Richardson (simonrichardson) wrote :
Changed in juju:
milestone: none → 2.8.1
assignee: nobody → Simon Richardson (simonrichardson)
status: Triaged → In Progress
status: In Progress → Fix Committed
Changed in juju:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.