Added the ability to create tokens #294

Merged
merged 6 commits into from Oct 7, 2014

Projects

None yet

2 participants

@laf
Member
laf commented Sep 29, 2014

No description provided.

@paulgear
Member
paulgear commented Oct 6, 2014

I have to say I'm not a huge fan of using JS to generate the password. Is there a way we could do it on the server side instead?

@laf
Member
laf commented Oct 6, 2014

I can change this so it auto-generates the token.

@laf
Member
laf commented Oct 6, 2014

Done, uses bin2hex(openssl_random_pseudo_bytes(16)) to generate a random token and puts this into the form (disabling the input).

@paulgear
Member
paulgear commented Oct 7, 2014

That means we don't need the index.php change or the custom.js file at all, correct?

@laf
Member
laf commented Oct 7, 2014

Also removed :)

@paulgear paulgear merged commit e6fe3be into librenms:master Oct 7, 2014
@laf laf deleted the unknown repository branch Apr 30, 2015
@laf laf pushed a commit to laf/librenms that referenced this pull request Nov 11, 2015
@dasmall dasmall Removing previously added style tags before adding new one.
Hopefully fixes #211 and #294.
1c3659c
@laf laf pushed a commit to laf/librenms that referenced this pull request Nov 11, 2015
@dsmorse dsmorse Merge remote-tracking branch 'dasmall/gridster.js.git/master'
* dasmall/gridster.js.git/master:
  Removing previously added style tags before adding new one. Hopefully fixes #211 and #294.
524ed2b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment