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

[request] krb5/1.18.3 #4102

Closed
SpaceIm opened this issue Jan 6, 2021 · 17 comments · Fixed by #16074
Closed

[request] krb5/1.18.3 #4102

SpaceIm opened this issue Jan 6, 2021 · 17 comments · Fixed by #16074

Comments

@SpaceIm
Copy link
Contributor

SpaceIm commented Jan 6, 2021

Package Details

Description Of The Library / Tool

Kerberos is a network authentication protocol. It is designed to provide strong authentication for client/server applications by using secret-key cryptography. A free implementation of this protocol is available from the Massachusetts Institute of Technology.

depends on OpenSSL

@madebr
Copy link
Contributor

madebr commented Jan 6, 2021

I have a wip recipe on my recipes_wip branch, if this is urgent.

@SpaceIm
Copy link
Contributor Author

SpaceIm commented Jan 7, 2021

Not urgent for me, mainly to improve cyrus-sasl recipe: #3466 (comment)

@ericLemanissier
Copy link
Contributor

it's also used by qt network module. do you need some help @madebr ? Maybe we could start with linux-only recipe ?

@ericLemanissier
Copy link
Contributor

it's also used by qt network module. do you need some help @madebr ? Maybe we could start with linux-only recipe ?

friendly ping @madebr

@jiggersplat
Copy link

+1

@friendlyanon
Copy link
Contributor

This is blocking #8901 from building on MacOS. @madebr can you give an update on that WIP recipe?

@ericLemanissier
Copy link
Contributor

@friendlyanon
Copy link
Contributor

That appears to be in a near complete condition. Hope to see it in CCI sooner than later!

@friendlyanon friendlyanon mentioned this issue Feb 16, 2022
4 tasks
tsondergaard added a commit to tsondergaard/conan-center-index that referenced this issue Jul 2, 2022
Extracted from [email protected]:madebr/conan-center-index.git
recipes_wip branch commit 17d10a4.

Fixes conan-io#4102
@ofiriluz
Copy link
Contributor

+1 on KRB5 Library

@kexianda
Copy link
Contributor

+1
krb5 is heavily used in production.

@xakod
Copy link
Contributor

xakod commented Nov 15, 2022

@madebr are there any chances to see krb5 in cci?

@madebr
Copy link
Contributor

madebr commented Nov 15, 2022

Fork https://github.com/madebr/conan-center-index/tree/recipes_wip/recipes/krb5/all, and make it conan v2 compatible.
I did some modifications to make it compile to static libraries, but I would remove it and only allow shared libraries.
Since this is a security focused library, such modifications might be dangerous.

@vlebourl
Copy link
Contributor

vlebourl commented May 3, 2023

+1
any news on this? Cheers

@xakod
Copy link
Contributor

xakod commented May 3, 2023

Hi, @vlebourl, please look at related pr error. Maybe you have any ideas how to fix it.

@fpoirotte
Copy link
Contributor

Hi,

The error is caused by the ksu program, which needs to be installed (and subsequently run) with the setuid bit set, to allow users to switch to a different Kerberos identity.

Apparently, in the CI environment, the packages are built with a non-privileged account, therefore the install command for that program fails.

You could patch the following line:
src/config/pre.in:187:INSTALL_SETUID=$(INSTALL) $(INSTALL_STRIP) -m 4755 -o root
to instead read:
src/config/pre.in:187:INSTALL_SETUID=$(INSTALL) $(INSTALL_STRIP) -m 755
This will allow the package to be built successfully, although the installed ksu program will be broken.

To fix it completely, end-users would have to install the package as root, which seems unreasonable in most situations.

@xakod
Copy link
Contributor

xakod commented Feb 12, 2024

@fpoirotte what do you think about removing ksu from cci package?

@fpoirotte
Copy link
Contributor

From my point of view, it would be fine, but maybe someone more accustomed with CCI could voice in on the subject.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
10 participants