Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Resources support #368

Merged
merged 1 commit into from
Oct 8, 2019
Merged

Conversation

smortex
Copy link
Contributor

@smortex smortex commented Sep 30, 2019

Tomcat 8 introduced Resources and some settings where moved arround:

<!-- Tomcat 7: -->
<Context allowLinking="true" />

<!-- Tomcat 8: -->
<Context>
  <Resources allowLinking="true" />
</Context>

This commit introduce a new tomcat::config::server::resources defined type which allows to configure such settings:

tomcat::config::server::resources { 'app resources':
  parent_host           => 'localhost',
  parent_context        => '/opt/app',
  additional_attributes => {
    allowLinking => true,
  },
}

Tomcat 8 introduced [Resources] and some settings where moved arround:

```
<!-- Tomcat 7: -->
<Context allowLinking="true" />

<!-- Tomcat 8: -->
<Context>
  <Resources allowLinking="true" />
</Context>
```

This commit introduce a new tomcat::config::server::resources defined
type which allows to configure such settings:

```
tomcat::config::server::resources { 'app resources':
  parent_host           => 'localhost',
  parent_context        => '/opt/app',
  additional_attributes => {
    allowLinking => true,
  },
}
```

[Resources]: https://tomcat.apache.org/tomcat-8.0-doc/config/resources.html
@michaeltlombardi michaeltlombardi merged commit 59a438e into puppetlabs:master Oct 8, 2019
@smortex smortex deleted the resources branch October 8, 2019 18:26
@smortex smortex restored the resources branch October 8, 2019 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants