nypd / soil-and-pimp-live-batch Goto Github PK

Also see if you can enhance these try catches with some more useful information

Should make the email a bit more helpful

Would be great to store the various updates the event has over time

The results from buildHtmlEventMarkup needs to get stored in its own variable or else the batch app starts appending everyone's email to one email.

Make them more flashy

This should resolve the issue with duplicate events

Fixes

#12 Fixed issue where unsubscribe links were not being reset for each email. Mega oops

From GitHub's fancy new vulnerability detection:

https://nvd.nist.gov/vuln/detail/CVE-2018-7489

Fixes

• #17 Fixed improper HTML syntax in the mail message
• #16 Logging errors properly rather than "system outing" them
• #19 Change the JSON url to https

Changes

• #11 Only send emails to verified addresses

spring-boot-starter > 2.0.1.RELEASE
spring-batch-core > 4.0.1.RELEASE

• Remove spring.version property
• Make sure spring-boot-starter-data-jpa uses the spring pom.xml property

https://nvd.nist.gov/vuln/detail/CVE-2019-3774

Needs to wait for the web-app portion of this task

We be getting errors now ever since they upgrades their site.

http://www.simplejavamail.org/#/debugging

Fixes

• #21 Upgraded Ms. Jackson to 2.9.8 to fix security issue

Minor

• Updated simple java mail and mysql connector to match web-app versions

Should be simple-ish doing so on the server

When submitting their email, the current browsers locale should be saved as well and stored in the email_subscriptions table.

Then the batch can translate the events on the fly before sending emails.

Should prob store the translated text temporarily.