Not registering the Environment variables

[rayman22201]

I set the environment variables as per the documentation, but Strider seems to be ignoring them.

I used the following environment variables:
PLUGIN_GITHUB_APP_ID
PLUGIN_GITHUB_APP_SECRET
PLUGIN_GITHUB_HOSTNAME

I don't see any references to those variables in the code.
Is the Strider extension loader is supposed to auto-load environment variables into the config object? 😖

For now, I made a fork with the following patch (read hack) that seems to work.
It explicitly looks for those environment vars in the config object declaration:
https://github.com/Galavantier/strider-github/blob/master/lib/webapp.js#L11-L13

I found this original issue that solved the problem, but it doesn't seem to be working now: #22

Any ideas about what is going on?

My patch seems to be working, but I would really prefer to help get the official repo working instead of maintaining a fork 😄

Thanks so much

[knownasilya]

Can you try master.

[rayman22201]

I did a pull from master. It does not work. :-(

If it helps. I am using a docker container that is pulling the master branch of Strider itself, as well as pulling master from this repo.

-------------------- update ---------------------
I put some debug statements in the code to see if I could garner any more info.

It looks like the APP ID and APP SECRET are correctly set, but the hostname is not.
It is still set to localhost.

[knownasilya]

Do you have SERVER_NAME set, as per the instructions here?

[rayman22201]

Ah ha! I set the SERVER_NAME environment variable, and that worked. 👍 😄

is PLUGIN_GITHUB_HOSTNAME used at all?

Thanks so much for your help.

[knownasilya]

Maybe not.. I'll look through the code when I get a chance, and if it's not we should remove it from the README.

[trein]

Hi @knownasilya and @rayman22201,

I'm facing a similar problem here. I'm using the plugin version 1.3.2, but I'm still getting localhost (http://localhost:3000/auth/github/callback) when adding a new account on account settings.

Here is my upstart script:

start on mounted MOUNTPOINT=/
stop on shutdown

env STRIDER_HOME=/opt/strider
env SERVER_NAME=http://myurl.com:1234
env PORT=1234
env DB_URI=mongodb://localhost:27017/strider-foss

script
    chdir $STRIDER_HOME
    exec npm start 1>>/opt/strider/log/stdout.log 2>>/opt/strider/log/stderr.log
    emit strider_running
end script

Thanks,

[knownasilya]

@trein try removing the port from SERVER_NAME.

[trein]

Thanks for the reply @knownasilya. I removed the port, but the issue persists.

[knownasilya]

@trein not sure, but your example has a typo: http:/myurl.., there is only one slash.

[trein]

@knownasilya sorry about that. I had to mask the real URL. It's just a typo on the comment. I'm going to fix it. On /etc/init/strider.conf it is correct.

[knownasilya]

Are any of the variables passing to strider, like the port? If you can confirm that, we can try other things.

[trein]

@knownasilya No. That's all that is being passed.

[knownasilya]

What I meant, is that in your example you show setting PORT, can you confirm that Strider is running on that port (via netstat or something)?

If you aren't setting it, could you, just as a test.

[trein]

@knownasilya Yes, it is running on port 1234.