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

update the way we retrieve the proxy template in order to offer the p… #156

Conversation

perezdf
Copy link
Member

@perezdf perezdf commented Oct 7, 2021

…ossibility url

@@ -6,4 +6,4 @@
# - a CoatRack build requires some mail configuration parameters to be set
# - the configuration values in this file are just dummy/fallback values for development builds

mvn clean package -Dygg.mail.sender.user="" -Dygg.mail.sender.password="" -Dygg.mail.server.url="" -Dygg.mail.server.port=0 -Dygg.mail.sender.from=""
mvn clean package -Dygg.mail.sender.user="" -Dygg.mail.sender.password="" -Dygg.mail.server.url="" -Dygg.mail.server.port=0 -Dygg.mail.sender.from="" -Dygg.proxy.executable-jar.template.url="" -Dygg.proxy.executable-jar.generated.path=""
Copy link
Contributor

Choose a reason for hiding this comment

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

Wouldn't it be better/easier to move these fallback parameter values into application.yml?

Copy link
Member Author

Choose a reason for hiding this comment

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

we will achive these and other movement of variable in a more and specific issue

Copy link
Member Author

Choose a reason for hiding this comment

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

@@ -33,5 +33,5 @@ if [ ! -z $1 ]; then
VM_ARG_SPRING_PROFILE="-Dspring.profiles.active=$1"
fi

java -jar $VM_ARG_SPRING_PROFILE -Dygg.mail.sender.user=$2 -Dygg.mail.sender.password=$3 $COATRACK_JAR_FILE
java -jar $VM_ARG_SPRING_PROFILE -Xdebug -Xrunjdwp:transport=dt_socket,address=8000,server=y,suspend=y -Dygg.mail.sender.user=$2 -Dygg.mail.sender.password=$3 $COATRACK_JAR_FILE
Copy link
Contributor

Choose a reason for hiding this comment

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

Does this mean we want to switch on remote debugging all the time? Maybe have a separate profile or start script for that purpose?

Besides, I read that instead of -Xdebug and -Xrunjdwp it would be more performant to use the parameter -agentlib:jdwp, see: https://stackoverflow.com/questions/138511/what-are-java-command-line-options-to-set-to-allow-jvm-to-be-remotely-debugged

Francisco Pérez and others added 5 commits October 8, 2021 13:57
…omProxyFileGeneratorService.java

Co-authored-by: gr-hovest-atb <gr-hovest@atb-bremen.de>
…omProxyFileGeneratorService.java

Co-authored-by: gr-hovest-atb <gr-hovest@atb-bremen.de>
…omProxyFileGeneratorService.java

Co-authored-by: gr-hovest-atb <gr-hovest@atb-bremen.de>
…omProxyFileGeneratorService.java

Co-authored-by: gr-hovest-atb <gr-hovest@atb-bremen.de>
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

4 participants