Token bug in converter.m

Ouverte
  • Quelque chose ne va pas ?
#35 0 commentaires 0 réactions 1 personne assignée Voir sur GitHub

@k-deeley y travaille déjà.

Depuis le 21/10/2026.

  • #37 par @k-deeley — ouverte

Évaluation

Cette issue n'a pas encore été évaluée.

Description

bug

Both the GitHub and GitLab cases pass "" as the default token (lines 21, 25), but the GitHub.m and GitLab.m constructors expect missing to skip authentication. So even when no token env var is set, "" gets sent as the auth header and the API returns 401. The unauthenticated GitHub path should work but can't because of this.

Reproduction Steps:
  1. Install DocMaker v0.8.0 (MATLAB_DocMaker.mltbx from the GitHub release) in MATLAB:
>> matlab.addons.install("MATLAB_DocMaker.mltbx")
  1. Make sure no token is set anywhere:
>> setenv("DOCMAKER_GITLAB_HOSTNAME", "")
>> setenv("DOCMAKER_GITLAB_TOKEN", "")
  1. Minimal repro:
>> setenv("DOCMAKER_CONVERTER", "GitHub")
>> c = docmaker.converter();
>> c.Token % Expected <missing>

ans = 

    ""

>> c.md2xml("# Hello")
Error using docmaker.GitHub/md2xml (line 80)
[HTTP/1.1 401 Unauthorized] Bad credentials
Langage dominant
MATLAB
Étoiles
10
Forks
3
Aucune PR mergée en 30 j

Préparer son environnement

Par où commencer

  1. Lisez l'issue en entier, puis le guide de contribution du projet.
  2. Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
  3. Forkez le dépôt et travaillez sur une branche.
  4. Ouvrez une pull request qui référence le numéro de l'issue.

Autres issues de mathworks/docmaker

Toutes les issues de mathworks/docmaker

Issues similaires

Plus d'issues MATLAB

Recevez les nouvelles issues par e-mail

Un résumé court des issues GitHub adaptées aux débutants.