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

Major update, courtesy HEAT, LLC #2

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

Major update, courtesy HEAT, LLC #2

wants to merge 14 commits into from

Conversation

EmperorArthur
Copy link

Hello,

HEAT, LLC is an environmental monitoring company based in Knoxville, TN. They have generously allowed me to upstream my work with your library. This library has been extremely helpful and saved me quite a bit of time, and I did not want my work to drift from what you all are using.

I am sorry for the long list of changes, and can break it up into multiple requests if you would like. This was merely the first time I was given the go ahead to upstream this code.

Major Feature Changes

  • The library now works with AWS PHP SDK version 3 (required implementing SHA-256 Hashing)
  • Backwards compatible method of disabling the SHA-256 Hashing for use with Python's Boto 3 Library.
  • The library now uses JavaScript classes (done via an automated conversion process, so it's not perfect)
  • Minor JS cleanups
  • Changed the composer.json license to MIT to match the LICENSE file
  • Added HEAT as a contributor.

Arthur Moore added 13 commits September 18, 2019 14:46
Updated the LICENSE file
8963057
Updated Composer with more requirements
ea4daaa 
These were already required, but were not in the compoer.json file.
Also added a composer.lock file.

The composer license did not match the License file, so updated that as
well.

Added myself as an Author.
Add a gitignore
06a81d8
Renamed index so it can be found automatically by the built in PHP Se…
361023a 
…rver
Updated the code to use the latest AWS API
46caea3
Don't shadow a protected language keyword
daa6e74
Use Intellij's "Convert to Class feature" on the two classes
7c9cabf
Minor javascript cleanup
59aab84
More JS Cleanups
4d4c6e3
Comment out a debug message
38c6a31
Removed some unused parameters for the ajax call
62a7e93
Add the ability to disable the Sha256 feature
a2e78b6 
Currently this requires editing the javascript, and the server requires it.
Some AWS Libraries don't actually use it, and it costs quite a bit of CPU.
Properly fail instead of hanging if a chunk fails to upload
3212d18
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.
1 participant
@EmperorArthur