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

Patch #13 works for us.
Patch #15 using deprecated API and breaks the /admin/config/openai/chatgpt page.

Patch #13 tested and works as expected.
Created MR !100

mariab → changed the visibility of the branch bugfix/3465426-increase-maxlength to hidden.

Patch #15 is a new feature and provides extra functionality which is out of scope. So, I opened a new issue for this.
https://www.drupal.org/project/openai/issues/3493573#comment-15899026 ✨ Integration with key module Active

I can't turn https://git.drupalcode.org/project/openai/-/merge_requests/98 to "ready" state but it does what it promises, related to the specific issue description. I set it as RTBC.

MR #98 resolved this issue for me.

I'm also happy with this patch.
+1 for getting it into a new release, please.

Patch #13 but only after I uninstalled/installed the module and added the new key again. Using Drupal 10.3.11. Patch #14 didn't work for me at all

@danrod, based on the change of the MR or the patch in #13, I can't guess why a reinstallation of the module would be required. A cache clear wouldn't suffice?

@vensires I just applied the patch #13 on a clean Drupal 10 installation and I had the same issue, these were the steps.

1. Installed Drupal 10.3.11
2. Installed module OpenAI
3. Added new OpenAI keys then went to /admin/config/openai/chatgpt and got the same error
4. Applied patch #13, cleared cache, had the same issue.
5. Installed/uninstalled module, didn't have any issues after that.

Perhaps I'm missing another step?

I don't doubt you but is does seem strange to me... It's just a simple maxlength attribute change. I also checked the schema.yml just in case and there isn't anything related in there either. Something is escaping us...
Setting this to "Needs work" until we find this and fix this.

@vensires I think setting this issue to "Needs work" is a bit too mucho, I'll do some extra tests and post a video later today, I am sure I skipped something.

@vensires I tested this patch again on a clean D11 installation, I installed the openapi module, enabled the modules, applied the patch then cleared cache and it worked this time, but I had to click on "Save configuration" twice for some reason.

Thank you for the test @danrod. I can't personally understand why the double submission or the clear cache is required; I really think it's RTBC but let's keep it under "Needs Review" for a while more.

Hi everyone,

I fixed adding #maxlength as 164 for the field api_key on the ApiSettingsForm class, maybe should be higher (not sure) but that did the trick for me.

Hi everyone,

I fixed adding #maxlength as 164 for the field api_key on the ApiSettingsForm class, maybe should be higher (not sure) but that did the trick for me.