Skip to content
This repository has been archived by the owner on Nov 15, 2022. It is now read-only.

Fix for Microsoft login #5

Open
LAK132 opened this issue Dec 26, 2016 · 1 comment · May be fixed by #7
Open

Fix for Microsoft login #5

LAK132 opened this issue Dec 26, 2016 · 1 comment · May be fixed by #7

Comments

@LAK132
Copy link

LAK132 commented Dec 26, 2016

Add a second username slot in the config file
eg:
"skype_username" : "asdf@live.com", //login username
"live_username" : "live:asdf_123", //username that is returned by messages or blank if not using Microsoft account

Compare the receiving username against both:

var spypeIsNotSender = (message.resources.from.toLowerCase().indexOf(config.skype_username.toLowerCase()) === -1) && (message.resources.from.toLowerCase().indexOf(config.live_username.toLowerCase()) === -1)

Viola, bot no longer replies to itself

@findel
Copy link
Owner

findel commented Jan 17, 2017

Hey! I don't think that I knew this was an issue.

If you would like to submit this as a pull request I would be happy to accept it.

(Sorry for the slow reply, email notifications seem to have not come through for issues here)

Thanks!

@maxjoehnk maxjoehnk linked a pull request Aug 28, 2017 that will close this issue
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants