Skip to content
This repository has been archived by the owner on May 4, 2021. It is now read-only.

add composer support #8

Open
kaystrobach opened this issue Apr 23, 2014 · 24 comments · May be fixed by #23
Open

add composer support #8

kaystrobach opened this issue Apr 23, 2014 · 24 comments · May be fixed by #23

Comments

@kaystrobach
Copy link

please add a valid composer.json and register your package at packagist

@roelmonnens
Copy link

+1

5 similar comments
@jbillen
Copy link

jbillen commented Sep 24, 2014

+1

@gvanbeck
Copy link

+1

@JurRutten
Copy link

+1

@ymaerschalck
Copy link

+1

@jeroen-cornelissen
Copy link

+1

@jas4711
Copy link
Contributor

jas4711 commented Sep 24, 2014

We are cleaning up this project here which will include all of that:

https://github.com/Yubico/yubico-php-client

Feedback on direction here is very much appreciated. We really want to drop the PEAR.php stuff and modernize the library a bit. We are worried with backwards compatibility, though, hence starting a separate project for this.

If you have experience writing PHP modules, now is a perfect time to help and shape the next generation of this library.

/Simon

@Potherca
Copy link

@jas4711 Should suggestions and concerns be raised here or in the new project repo?
I may have a few sugestions in regards to "modernize"

@jas4711
Copy link
Contributor

jas4711 commented Sep 30, 2014

I think we really want this project (php-yubico) to be backwards compatible, and to remain a "drop-in" upgrade path for all the people using current or older versions of the project.

Full backwards compatibility probably means we can't clean up this project as much as we want too -- hence a new project.

So if your concern is about "modernization", let's take those in the new project, but if it is about functionality in the old client, let's take it here. Note the issue in the new project about reusing another third-party php library instead -- we might end up working on that one instead of finishing the new project.

Sorry about confusion here -- my main message is that we are happy to get help here, and we are open to most ideas (that are sane :)).

/Simon

@AlexGx
Copy link

AlexGx commented Oct 22, 2014

+1 for composer support

@Ardakilic
Copy link

+1 , I'd love to see an official package on Composer.

@tsmgeek
Copy link

tsmgeek commented Jul 5, 2016

+1

1 similar comment
@reksc
Copy link

reksc commented Nov 21, 2016

+1

@overedge
Copy link

+666

@JurRutten
Copy link

+666 indeed, what is taking so long?
You are a commercial company, we are recommending your product to our users, please get of your ass and do something...

@libregeek
Copy link

libregeek commented Jul 27, 2017

+1
Please add composer support so that we don't need to worry about upgrades.

@tvdijen
Copy link

tvdijen commented Dec 18, 2017

+1

1 similar comment
@scarl555
Copy link

+1

@brother
Copy link

brother commented Jan 10, 2019

We are cleaning up this project here which will include all of that:

https://github.com/Yubico/yubico-php-client

Any updates to brag about?
Four years have passed, the above mentioned repo is gone (or at least not public).

kaystrobach added a commit to kaystrobach/php-yubico that referenced this issue Jan 10, 2019
@kaystrobach kaystrobach linked a pull request Jan 10, 2019 that will close this issue
@kaystrobach
Copy link
Author

how about that? - took min 2 min to generate one ...

@kaystrobach
Copy link
Author

I think that project is dead ...

@brother
Copy link

brother commented Jan 10, 2019

I think I'll use https://github.com/enygma/yubikey instead. Covers what I need and is already in packagist...

@Elycin
Copy link

Elycin commented Jan 10, 2019

@brother I believe this is what I ended up using at the end of the day last year. Good luck on your development, although I have found that there is a cURL specific bug where the requests cannot be made on a windows environment.

@tvdijen
Copy link

tvdijen commented Mar 22, 2019

I can't believe a new release was made without taking composer-support into consideration :(

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging a pull request may close this issue.