Skip to content

Conversation

@joshiste
Copy link
Contributor

@joshiste joshiste commented Jan 6, 2015

closes #2137

@philwebb
Copy link
Member

philwebb commented Jan 6, 2015

Cool. We'll take a look at this for 1.3. I wonder if we should disable the endpoint if the logging.file property is not set?

@philwebb philwebb added this to the 1.3.0 milestone Jan 6, 2015
@joshiste
Copy link
Contributor Author

Now the logfile-endpoint is only added when logging.file or logging.path is set.
BTW CLA is 93620140928055712

@joshiste
Copy link
Contributor Author

joshiste commented Jun 2, 2015

I'm curious wether you gonna merge this into 1.3.0?

@wilkinsona
Copy link
Member

That's the plan – it's currently assigned to the 1.3.0.RC1 milestone.

@philwebb philwebb modified the milestones: 1.3.0.M2, 1.3.0.RC1 Jun 17, 2015
@philwebb philwebb closed this in 308a5ea Jun 24, 2015
philwebb added a commit that referenced this pull request Jun 24, 2015
* pr/2294:
  Add /logfile MVC actuator endpoint
@philwebb
Copy link
Member

Merged with a few tweaks. Primarily removing the "file download" prompt when hitting the endpoint.

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

An endpoint for logging

3 participants