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

Replacing the previous uuid library with our lua_uuid #695

Merged
merged 1 commit into from
Nov 10, 2015
Merged

Conversation

subnetmarco
Copy link
Member

This PR replaces https://github.com/Tieske/uuid with https://github.com/Mashape/lua-uuid.

The previous UUID library seemed to produce UUID conflicts that lead to errors like #659. The new library leverages libuuid which provides a more solid implementation.

When we decide to merge this PR, we need to update https://github.com/Mashape/kong-distributions to include a dependency to libuuid whenever the operating system doesn't already ship with it.

@thibaultcha
Copy link
Member

Side note: this might also fix #478, assuming the supposed cause of the issue is correct.

@subnetmarco
Copy link
Member Author

Updated with new version of lua_uuid. Ready to merge.

@subnetmarco subnetmarco added pr/ready (but hold merge) No more concerns, but do not merge yet (probably a conflict of interest with another PR or release) and removed status/needs review labels Nov 9, 2015
subnetmarco added a commit that referenced this pull request Nov 10, 2015
Replacing the previous uuid library with our lua_uuid
@subnetmarco subnetmarco merged commit fc9715a into next Nov 10, 2015
@subnetmarco subnetmarco deleted the chore/uuid branch November 10, 2015 02:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr/ready (but hold merge) No more concerns, but do not merge yet (probably a conflict of interest with another PR or release)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants