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

Use explicit urlencoded POST data #385

Merged
merged 1 commit into from
Apr 30, 2014

Conversation

ax3l
Copy link
Member

@ax3l ax3l commented Apr 30, 2014

  • avoids problems with signs like &
  • use python based json encoding

- avoids problems with signs like &
- use python based json encoding
@ax3l ax3l added bug and removed feature labels Apr 30, 2014
@ax3l ax3l added this to the Open Beta milestone Apr 30, 2014
# escape special characters
textJSON=`echo -n "$text" | python -c 'import json,sys; print json.dumps(sys.stdin.read())'`
postParams='{"action":"report","eventid":'$eventid',"result":"'$stateName'","output":'$textJSON'}'
echo -n "$postParams" > "$thisDir"lastPostParams.log
Copy link
Member

Choose a reason for hiding this comment

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

is $thisDir guaranteed to end on /?

Copy link
Member Author

Choose a reason for hiding this comment

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

yes its defined in new_commits.sh line 24

f-schmitt pushed a commit that referenced this pull request Apr 30, 2014
Use explicit urlencoded POST data
@f-schmitt f-schmitt merged commit 33b1e4b into ComputationalRadiationPhysics:dev Apr 30, 2014
@ax3l ax3l deleted the topic-encCompile branch April 30, 2014 12:14
psychocoderHPC pushed a commit to psychocoderHPC/picongpu that referenced this pull request Sep 7, 2017
048a3d3 Merge pull request ComputationalRadiationPhysics#394 from BenjaminW3/topic-events
f41d247 add more tests for events
86f5bf6 Merge pull request ComputationalRadiationPhysics#393 from BenjaminW3/topic-block-sync-test
5a36a6b add the missing block synchronization tests to the CI
8237771 Merge pull request ComputationalRadiationPhysics#392 from BenjaminW3/topic-unit-tests
6ad6226 add unit tests for events and rand
549733d Merge pull request ComputationalRadiationPhysics#373 from BenjaminW3/topic-stream-callback
462152a Merge pull request ComputationalRadiationPhysics#389 from BenjaminW3/topic-event-cpu-fix
f5bbb58 fix EventCpu deadlock
775c8b3 fixup doc [ci skip]
05663c7 update doc [ci skip]
439c98b Merge pull request ComputationalRadiationPhysics#385 from BenjaminW3/topic-fix-vs-2017_3
49ed005 support VS 2017.3
f1cfe26 add stream callback

git-subtree-dir: thirdParty/alpaka
git-subtree-split: 048a3d3d18d181e10a0e9033dcc6fe7d62f5884a
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug a bug in the project's code component: tools scripts, python libs and CMake
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants