New, long keys can't be stored properly

Thanks for flagging the issue. Few things here:

1. Can you say the length of your key? Text area probably not right, probably should be a text field still with a higher #size
2. Please create as a merge request for easier review and merge
3. If storage size in database table needs to change, needs update hook otherwise existing sites will have entity definition warnings in admin status report. Something like this https://agaric.coop/blog/change-text-field-maximum-length-drupal-8

.

The length of my key 132.

Rahaf Albawab → made their first commit to this issue’s fork.

Hi @scott_euser.
I'm facing the same issue, and my key length is also 132. I created a merge request; you can check it.

I tried #7 but no luck yet.

Instead of increasing the maxlength for the API key field, I recommend using the Key module for securely storing API keys. This allows flexibility and better security without worrying about arbitrary field length limits.

if (\Drupal::moduleHandler()->moduleExists('key')) {
  $key = \Drupal\key\Entity\Key::load('openai_api_key');
  $api_key = $key ? $key->getKeyValue() : '';
} else {
  $api_key = $this->config('openai.settings')->get('api_key');
}

Yep that's how we are doing it in the ai module which is slowly taking this over. https://www.drupal.org/project/ai →

Hope I'm not overstepping my boundaries here but I think a bump in priority is called for. This module is now not available to new projects or projects using a newer api key.

I've got one old key in my possession that's now in use on a handful of sites and that is not ideal, but for those without access to older keys this module is unusable at the moment.

Migrating to the new ai module is definitely the way to go in general but it's going to take a lot of time in many cases so we don't have a viable work around right now leaving some people in the dust.

same issue here. Newer API Key seems make error and all the function not working in openAI module.

Had the same issue, my API-key had 163 characters. Forcing the field to allow 255 characters fixed the issue in my case.
Patch attached.

Had the same issue, but our key was even longer and we use they key module anyway, so on top of increasing the field size to 256 I added a bit more logic to cover the use case for they key module as well.
Patch attached.

same issue here.
New/long API Key made error so the OpenAI module cant work as usually