Skip to content

Sample statistics and notifications server #114

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

mgaffigan
Copy link
Contributor

Closes #24 and #25. Alternative to #79. Connect Service is a sample implementation of a web service which records usage
of Mirth Server in ElasticSearch to allow Kibana Dashboards, and provides access to a notifications feed. The service implements
the operations called by the client in ConnectServiceUtil.java.

Testing may be accomplished by:

  1. Set URL_CONNECT_SERVER in ConnectServiceUtil.java to "http://localhost:3000"
  2. Update REPO_URL in connectservice/github.js to 'https://api.github.com/repos/nextgenhealthcare/connect/releases?per_page=10' to have interesting example data
  3. In the new connectservice directory, run docker-compose build followed by
    docker-compose up -d
  4. Build and run the server and client as normal. Observe notifications from github, and usage recorded to Elasticsearch (see http://localhost:5601/app/management/data/index_management/indices/)

Known issues:

  • Images seem to work intermittently with the webview - not sure this is related to the server, probably client-side. Seems inconsistent.

Screenshots:

Screenshot of notifications showing github releases
Screenshot showing notifications with working images
Screenshot of registration data
Screenshot of usage data

…sage

Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
@mgaffigan mgaffigan force-pushed the feature/add-sample-connectservice branch from c730277 to 5475d95 Compare June 2, 2025 04:51
@jonbartels jonbartels requested review from jonbartels, a team, kayyagari, gibson9583, ssrowe, kpalang, tonygermano and pacmano1 and removed request for a team June 2, 2025 15:43
Signed-off-by: Mitch Gaffigan <mitch.gaffigan@comcast.net>
@mgaffigan
Copy link
Contributor Author

Review notes from accidental merge in other PR:

from @jonbartels:

  • connectservice/.dockerignore
    Why a second .dockerignore? Does the parent directory ignore file apply here?
    • I'm viewing this as something of a "sample" folder that I would assume someone copies out of tree - as such, I was trying to make it self-contained.
  • connectservice/Readme.md
    OIE not Mirth :D
    • Updated.

from @kpalang:

  • connectservice/Dockerfile
    suggestion: I'd use a newer node, 22 is latest LTS.
    • Good catch. Updated.
  • connectservice/Dockerfile
    thought: If you have time and interest, look into the new hotness that is bun ;)
    • Mmm... bun. (still aiming for lowest-common denominator in samples)

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.

Implement sample notifications server
1 participant