forked from moqui/moqui-framework
-
Notifications
You must be signed in to change notification settings - Fork 0
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
Integration branch #6
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…e commons-io, jackson-databind, jsoup, slf4j, snakeyaml
…tes, should work on more databases and nicer to user as it now logs results
…ead of the deprecated Whitelist class, also fix some type safety warnings
…ostgres, allow more flexibility than the default 30 for these databases that support it
…oduct store setting type where the setting may have enumerated options instead of just a text value field, and other uses where a specific enum may have a related enum type
…bapp_allow_origins setting, compare by dot-separate segment instead of ends-with as a fallback that could be bypassed using a custom domain name with a prefix on an allowed one
… were not being handled
…ds, use in EntityServices.export#EntityDataSnapshot
…andleCors() method which has code refactored from the service() method
…-csv, jackson-databind, shiro-core/-web, and slf4j-api in moqui-util which was missed in a prior update when framework/build.gradle was updated
…le null location with empty/null return value rather than NPE
…integration configuration changes
…lds, and change audit logging so that newValueText and oldValueText are encrypted for encrypted fields
…code using Apache Commons IOUtils
…aused field is null then the job is not paused, only if set to a value other than 'N', makes more sense given the name of the field and resolves some confusion around its behavior since the change from equals 'Y' to not equals 'N'
…rText() allow isTemplateStr to be a default extension if no renderer found give location filename extension
Added new bodyParameters parameter to the reset#Password service.
…and send#EmailTemplate services to only send to email addresses with matching domains, or if not set still to any domain
…ds the header, defaults to false which sets/overrides the header by name; change all response.addHeader() to response.setHeader() because on review the behavior we generally want is to override any headers set
…d, at least via the EntityFacade; this is one of the more obvious use cases for this feature and will help prevent audit log tampering, except via direct SQL
… to remote_addr instead of using the default proxy_add_x_forwarded_for which appends to an existing X-Forwarded-For and that makes it vulnerable to a header spoof attack
…ormat directly for locale sensitive grouping and up to 12 decimal digits with min of zero for integers; with this change might be best to remove most format strings throughout apps
Add start component to addons and createComponent task to build.grade
…nes to .gitignore
Acme docker compose
…ion/x-www-form-urlencoded
…p that is incompatible with JSON is passed in - integer as key
…mework into integration-branch
… custom directory; introduced new bat file to run unit tests in one call
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.