Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Using popen instead of proc_open and trying to simplify things with curl #59

Closed
wants to merge 2 commits into from

Conversation

dannylamb
Copy link
Contributor

GitHub Issue: Part of Islandora/documentation#978

What does this Pull Request do?

Bypasses ApiXMiddleware and CmdExecuteService entirely and uses popen to simplify the number of streams we're managing. This is just for @seth-shaw-unlv to test. If it actually works, i'll update CmdExecuteService accordingly. Otherwise you can just ignore this.

Interested parties

@seth-shaw-unlv

@dannylamb
Copy link
Contributor Author

Closing for now since Islandora/Alpaca#58 does this trick and this just obfuscates things.

@dannylamb dannylamb closed this Mar 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant