Cleanup deprecated Sequence Token code

[ernestomi]

Issue #10 corrected a bug arising from the deprecation of nextSequenceToken. We now need to a) cleanup the code to remove useless DataAlreadyAcceptedException and InvalidSequenceTokenException catchs, the reading of uploadSequenceToken and its storage in self.next_sequence_token

Also we need to introduce a test with concurrent log events to the same stream

[ernestomi]

It seems that #13 has been caused by update from ticket #10, updating the tag of this issue to bug

[datitran]

Hmm not ideal to remove the version as we used it. I think you need to pin the boto version. It seems that older boto version fails due to validation error on the parameter.

[ernestomi]

Hi @datitran, sorry to remove it so suddenly, but it introduced an unintended breaking change for other users.

Thanks for the tip on pinning the boto version, I want to check if we can do something to tolerate both older and newer versions, and if not, we can require the higher version.

[datitran]

@ernestomi I tried the most recent boto version and the validation is still there. I think I tested it yesterday with any value in there and thought it also works with no value. This will work:

def send_log(self, timestamp, log_entry):
    #Send the message to AWS
    self.logs.put_log_events(logGroupName=self.log_group,
        logStreamName=self.log_stream,
        sequenceToken='ignore', # ignored by PutLogEvents
        logEvents=[{'timestamp': timestamp,'message': log_entry}])

[datitran]

Tested it on #13 you can also try it out yourself with his example

[ernestomi]

@datitran thanks! I will test it later tonight. I will also try to cleanup unnecesary code. Ill merge your fork into a new branch, update and then merge again into the main. Thanks again!