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.

Sign up for GitHub

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

Jump to bottom

Incorporate mabrek/alertlogic changes #63

Closed
wants to merge 14 commits into from
Closed

Incorporate mabrek/alertlogic changes #63

wants to merge 14 commits into from

Conversation

nalundgaard
Copy link

  • changes from mabrek/httpc
  • fix gen_tcp:recv 64M limit
  • make files UTF-8 compliant for Erlang 17/18+
  • handle transfer-encoding correctly

mabrek and others added 13 commits April 10, 2013 20:42
@mabrek
catch both 'error' and 'exit'
4bdb080
@mabrek
trap exits and save Pid in stop reason
65b0735
@mabrek
handle 'done' from unknown processes (after manager restart)
b1f0c26
@mabrek
Merge remote-tracking branch 'github.mabrek/catch-exit'
e9a576a
@mabrek
typo in case clause
6451992
@juantascon
fixed otp's gen_tcp:recv 64M limit
10deea5
@juantascon
Merge remote-tracking branch 'mabrek/master' into fix_64m
4e84a21
all files in ISO have been converted into UTF-8 (the reason was autho…
9cc26d3 
…r's name)
@juantascon
Merge pull request #2 from tdosoudil/dev
5ac17e5 
all files in ISO have been converted into UTF-8
@juantascon
Merge pull request #3 from alertlogic/dev
4b02ab9 
Dev
@akontsevoy
Ignore Content-Length in favor of Transfer-Encoding per standard
127ab4f
Merge pull request esl#4 from kontsevoy/fix_chunked_encoding
5f7bbfd 
Ignore Content-Length in favor of Transfer-Encoding per standards
@nalundgaard
Merge remote-tracking branch 'alertlogic-ghee/master' into alertlogic…
fdab321 
…-master

# Conflicts:
#	test/lhttpc_manager_tests.erl
#	test/lhttpc_tests.erl
@nalundgaard
Copy link
Author

@motobob @juantascon can you guys have a look at this? to merge this atop esl I had to patch up the startup conflicts in the unit tests. I favored the R15+ method (using application:ensure_all_started/1)

@juantascon
Copy link

this pullrq fixes issues #57 and #48

@nalundgaard
Copy link
Author

Oy, I made it to the wrong fork! This was meant to go to the public Alert Logic fork. I guess I will leave this PR here, but I will make another one to the correct place.

@nalundgaard
Update .travis.yml otp_releases
2d99a59 
Only support latest erlangs R16-19
@nalundgaard
Copy link
Author

I am going to go ahead and close this, actually. While these fixes do seem a good idea, given that this repo is abandoned, our company will just use a fork of it with these changes (and removal of legacy support for pre-R16 Erlangs). See alertlogic#2.

@motobob motobob deleted the alertlogic-master branch September 19, 2016 10:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@nalundgaard @juantascon @mabrek @akontsevoy