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

added web-deployable implementation #13

Closed
wants to merge 5 commits into from
Closed

added web-deployable implementation #13

wants to merge 5 commits into from

Conversation

acoburn
Copy link
Contributor

@acoburn acoburn commented Apr 13, 2015

This creates a web-deployable audit service based on the camel routes in audit-triplestore, mostly resolving: https://jira.duraspace.org/browse/FCREPO-1452

There is some (minor) cleanup that should still be done, such as:

The audit-triplestore-web/pom.xml file can be cleaned up a bit; also there are no tests to speak of (I'm hoping that someone with more experience with Tomcat can do that -- basically just a sanity check). Also, are the jboss-related maven configs necessary? (feel free to remove them)

Finally, the jms configuration (in applicationContext.xml) should be made to be configurable from a system value (or property placeholder)

</configuration>
</plugin>

<!-- plugin to deploy to JBoss AS with: mvn jboss-as:deploy (you can also do redeploy/undeploy) -->
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed, let's not introduce JBoss AS deployment.

@awoods
Copy link

awoods commented Apr 14, 2015

Very nice. Quickly starting:

  • fuseki
  • fcrepo4, and
  • this camel webapp

immediately revealed audit events via SPARQL-Queries. 👍

@acoburn
Copy link
Contributor Author

acoburn commented Apr 14, 2015

Any opinions on the default value for the Audit Event BaseURI? I've got http://example.com/event for now.

@escowles
Copy link
Contributor

http://example.com/event seems good to me

@acoburn
Copy link
Contributor Author

acoburn commented Apr 14, 2015

@awoods @escowles pushed a new commit addressing most of this -- we still have log4j in there and there are, as of yet, no tests for the web deployable project. Those will be coming, but maybe not until later today.

@awoods
Copy link

awoods commented Apr 15, 2015

Merged as three squashed commits. Starting with:
2b5ce4a

@awoods awoods closed this Apr 15, 2015
@acoburn acoburn deleted the fcrepo-1452 branch August 17, 2015 13:11
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.

None yet

3 participants