re-implemented authorization

Signed-off-by: Stephan Richter <s.richter@srsoftware.de>

de.srsoftware.oidc.api/src/main/java/de/srsoftware/oidc/api/AuthorizationService.java

@@ -1,17 +0,0 @@
-/* © SRSoftware 2024 */
-package de.srsoftware.oidc.api;
-
-import java.time.Instant;
-import java.util.List;
-import java.util.Optional;
-
-public interface AuthorizationService {
-	AuthorizationService addCode(Client client, User user, String code);
-	AuthorizationService authorize(Client client, User user, Instant expiration);
-	boolean	     isAuthorized(Client client, User user);
-	List<User>	     authorizedUsers(Client client);
-	List<Client>	     authorizedClients(User user);
-	AuthorizationService revoke(Client client, User user);
-
-	Optional<Authorization> forCode(String code);
-}

de.srsoftware.oidc.api/src/main/java/de/srsoftware/oidc/api/AuthorizedScope.java

@@ -3,5 +3,5 @@ package de.srsoftware.oidc.api;
 
 import java.time.Instant;
 
-public record Authorization(String clientId, String userId, Instant expiration) {
+public record AuthorizedScope(String scope, Instant expiration) {
 }

de.srsoftware.oidc.api/src/main/java/de/srsoftware/oidc/api/ClaimAuthorizationService.java

@@ -0,0 +1,14 @@
+/* © SRSoftware 2024 */
+package de.srsoftware.oidc.api;
+
+import java.time.Instant;
+import java.util.Collection;
+import java.util.List;
+import java.util.Set;
+
+public interface ClaimAuthorizationService {
+	public record AuthResult(List<AuthorizedScope> authorizedScopes, Set<String> unauthorizedScopes, String authCode) {
+	}
+	AuthResult	          getAuthorization(User user, Client client, Collection<String> scopes);
+	ClaimAuthorizationService authorize(User user, Client client, Collection<String> scopes, Instant expiration);
+}

de.srsoftware.oidc.api/src/main/java/de/srsoftware/oidc/api/Constants.java

@@ -6,6 +6,7 @@ public class Constants {
 	public static final String APP_NAME      = "LightOIDC";
 	public static final String AUTH_CODE     = "authorization_code";
 	public static final String AUTHORIZATION = "Authorization";
+	public static final String AUTHORZED     = "authorized";
 	public static final String BEARER        = "Bearer";
 	public static final String CAUSE         = "cause";
 	public static final String CLIENT_ID     = "client_id";
@@ -14,6 +15,7 @@ public class Constants {
 	public static final String CONFIG_PATH   = "LIGHTOIDC_CONFIG_PATH";
 	public static final String CONFIRMED     = "confirmed";
 	public static final String DAYS	         = "days";
+	public static final String EXPIRATION    = "expiration";
 	public static final String EXPIRES_IN    = "expires_in";
 	public static final String GRANT_TYPE    = "grant_type";
 	public static final String ID_TOKEN      = "id_token";

de.srsoftware.oidc.backend/src/main/java/de/srsoftware/oidc/backend/ClientController.java

@@ -3,26 +3,23 @@ package de.srsoftware.oidc.backend;
 
 import static de.srsoftware.oidc.api.Constants.*;
 import static de.srsoftware.oidc.api.Permission.MANAGE_CLIENTS;
-import static de.srsoftware.utils.Strings.uuid;
-import static java.lang.System.Logger.Level.ERROR;
 import static java.net.HttpURLConnection.*;
 
 import com.sun.net.httpserver.HttpExchange;
 import de.srsoftware.oidc.api.*;
 import java.io.IOException;
-import java.time.Duration;
 import java.time.Instant;
-import java.util.Arrays;
+import java.time.temporal.ChronoUnit;
-import java.util.HashSet;
+import java.util.*;
-import java.util.Map;
+import java.util.stream.Collectors;
 import org.json.JSONObject;
 
 public class ClientController extends Controller {
-	private static final System.Logger LOG = System.getLogger(ClientController.class.getSimpleName());
+	private static final System.Logger      LOG = System.getLogger(ClientController.class.getSimpleName());
-	private final AuthorizationService authorizations;
+	private final ClaimAuthorizationService authorizations;
-	private final ClientService	   clients;
+	private final ClientService	        clients;
 
-	public ClientController(AuthorizationService authorizationService, ClientService clientService, SessionService sessionService) {
+	public ClientController(ClaimAuthorizationService authorizationService, ClientService clientService, SessionService sessionService) {
 		super(sessionService);
 		authorizations = authorizationService;
 		clients        = clientService;
@@ -30,33 +27,58 @@ public class ClientController extends Controller {
 
 
 	private boolean authorize(HttpExchange ex, Session session) throws IOException {
-		var user  = session.user();
+		var user = session.user();
-		var json  = json(ex);
+		var json = json(ex);
-		var scope = json.getString(SCOPE);
+		for (String param : List.of(SCOPE, RESPONSE_TYPE, CLIENT_ID, REDIRECT_URI)) {
-		if (!Arrays.asList(scope.split(" ")).contains(OPENID)) return sendContent(ex, HTTP_BAD_REQUEST, Map.of(ERROR, "openid scope missing in request"));
+			if (!json.has(param)) return badRequest(ex, "Missing required parameter \"%s\"!".formatted(param));
+		}
+		var scopes = toList(json, SCOPE);
+		if (!scopes.contains(OPENID)) return badRequest(ex, "This is an OpenID Provider. You should request \"openid\" scope!");
+		var responseTypes = toList(json, RESPONSE_TYPE);
+		for (var responseType : responseTypes) {
+			switch (responseType) {
+				case ID_TOKEN:
+				case TOKEN:
+					return sendContent(ex, HTTP_NOT_IMPLEMENTED, "Response type \"%s\" currently not supported".formatted(responseType));
+				case CODE:
+					break;
+				default:
+					return badRequest(ex, "Unknown response type \"%s\"".formatted(responseType));
+			}
+		}
+		if (!responseTypes.contains(CODE)) return badRequest(ex, "Sorry, at the moment I can only handle \"%s\" response type".formatted(CODE));
 
 		var clientId  = json.getString(CLIENT_ID);
-		var redirect  = json.getString(REDIRECT_URI);
 		var optClient = clients.getClient(clientId);
 		if (optClient.isEmpty()) return badRequest(ex, Map.of(CAUSE, "unknown client", CLIENT_ID, clientId));
-		var client = optClient.get();
+		var client   = optClient.get();
+		var redirect = json.getString(REDIRECT_URI);
 		if (!client.redirectUris().contains(redirect)) return badRequest(ex, Map.of(CAUSE, "unknown redirect uri", REDIRECT_URI, redirect));
+		var state = json.has(STATE) ? json.getString(STATE) : null;
 
 		client.nonce(json.has(NONCE) ? json.getString(NONCE) : null);
+		if (json.has(AUTHORZED)) {
+			var authorized = json.getJSONObject(AUTHORZED);
+			var days       = authorized.getInt("days");
+			var list       = new ArrayList<String>();
+			authorized.getJSONArray("scopes").forEach(scope -> list.add(scope.toString()));
+			authorizations.authorize(user, client, list, Instant.now().plus(days, ChronoUnit.DAYS));
+		}
 
-		if (!authorizations.isAuthorized(client, session.user())) {
+		var authResult = authorizations.getAuthorization(user, client, scopes);
-			if (json.has(DAYS)) {
+		if (!authResult.unauthorizedScopes().isEmpty()) {
-				var days       = json.getInt(DAYS);
+			return sendContent(ex, Map.of("unauthorized_scopes", authResult.unauthorizedScopes(), "rp", client.name()));
-				var expiration = Instant.now().plus(Duration.ofDays(days));
-				authorizations.authorize(client, user, expiration);
-			} else {
-				return sendContent(ex, Map.of(CONFIRMED, false, NAME, client.name()));
-			}
 		}
-		var state = json.getString(STATE);
+		var authoriedScopes = authResult.authorizedScopes().stream().map(AuthorizedScope::scope).collect(Collectors.joining(" "));
-		var code  = uuid();
+		var result	    = new HashMap<String, String>();
-		authorizations.addCode(client, session.user(), code);
+		result.put(SCOPE, authoriedScopes);
-		return sendContent(ex, Map.of(CONFIRMED, true, CODE, code, REDIRECT_URI, redirect, STATE, state));
+		result.put(CODE, authResult.authCode());
+		if (state != null) result.put(STATE, state);
+		return sendContent(ex, result);
+	}
+
+	private List<String> toList(JSONObject json, String key) {
+		return Arrays.asList(json.getString(key).split(" "));
 	}
 
 	private boolean deleteClient(HttpExchange ex, Session session) throws IOException {
@@ -86,7 +108,7 @@ public class ClientController extends Controller {
 	@Override
 	public boolean doPost(String path, HttpExchange ex) throws IOException {
 		var optSession = getSession(ex);
-		if (optSession.isEmpty()) return sendEmptyResponse(HTTP_UNAUTHORIZED, ex);
+		if (optSession.isEmpty()) return sendContent(ex, HTTP_UNAUTHORIZED, "No authorized!");
 
 		// post-login paths
 		var session = optSession.get();

de.srsoftware.oidc.backend/src/main/java/de/srsoftware/oidc/backend/TokenController.java

@@ -2,33 +2,29 @@
 package de.srsoftware.oidc.backend;
 
 import static de.srsoftware.oidc.api.Constants.*;
-import static de.srsoftware.utils.Optionals.nullable;
 import static java.lang.System.Logger.Level.*;
-import static java.net.HttpURLConnection.HTTP_BAD_REQUEST;
+import static java.net.HttpURLConnection.HTTP_NOT_IMPLEMENTED;
 
 import com.sun.net.httpserver.HttpExchange;
 import de.srsoftware.oidc.api.*;
 import java.io.IOException;
-import java.net.URLDecoder;
-import java.nio.charset.StandardCharsets;
 import java.util.*;
 import java.util.stream.Collectors;
 import org.jose4j.jwk.PublicJsonWebKey;
 import org.jose4j.jws.JsonWebSignature;
 import org.jose4j.jwt.JwtClaims;
 import org.jose4j.lang.JoseException;
-import org.json.JSONObject;
 
 public class TokenController extends PathHandler {
 	public record Configuration(String issuer, int tokenExpirationMinutes) {
 	}
-	private final ClientService	   clients;
+	private final ClientService	        clients;
-	private final AuthorizationService authorizations;
+	private final ClaimAuthorizationService authorizations;
-	private final UserService	   users;
+	private final UserService	        users;
-	private final KeyManager	   keyManager;
+	private final KeyManager	        keyManager;
-	private Configuration	   config;
+	private Configuration	        config;
 
-	public TokenController(AuthorizationService authorizationService, ClientService clientService, KeyManager keyManager, UserService userService, Configuration configuration) {
+	public TokenController(ClaimAuthorizationService authorizationService, ClientService clientService, KeyManager keyManager, UserService userService, Configuration configuration) {
 		authorizations	= authorizationService;
 		clients	= clientService;
 		this.keyManager = keyManager;
@@ -53,7 +49,8 @@ public class TokenController extends PathHandler {
 	private boolean provideToken(HttpExchange ex) throws IOException {
 		var map = deserialize(body(ex));
 		// TODO: check 	data, → https://openid.net/specs/openid-connect-core-1_0.html#TokenEndpoint
-
+		return sendEmptyResponse(HTTP_NOT_IMPLEMENTED, ex);
+		/*
 		var grantType = map.get(GRANT_TYPE);
 		if (!AUTH_CODE.equals(grantType)) return sendContent(ex, HTTP_BAD_REQUEST, Map.of(ERROR, "unknown grant type", GRANT_TYPE, grantType));
 
@@ -87,7 +84,7 @@ public class TokenController extends PathHandler {
 		response.put(EXPIRES_IN, 3600);
 		response.put(ID_TOKEN, jwToken);
 		LOG.log(DEBUG, jwToken);
-		return sendContent(ex, response);
+		return sendContent(ex, response);*/
 	}
 
 	private String createJWT(Client client, User user) {
@@ -120,12 +117,12 @@ public class TokenController extends PathHandler {
 		claims.setSubject(user.uuid());	  // the subject/principal is whom the token is about
 		claims.setAudience(client.id());
 		claims.setExpirationTimeMinutesInTheFuture(config.tokenExpirationMinutes);  // time when the token will expire (10 minutes from now)
-		claims.setIssuedAtToNow();	         // when the token was issued/created (now)
+		claims.setIssuedAtToNow();			            // when the token was issued/created (now)
 
 		claims.setClaim("client_id", client.id());
 		claims.setClaim("email", user.email());  // additional claims/attributes about the subject can be added
 		client.nonce().ifPresent(nonce -> claims.setClaim(NONCE, nonce));
-		claims.setGeneratedJwtId();	         // a unique identifier for the token
+		claims.setGeneratedJwtId();  // a unique identifier for the token
 		return claims;
 	}
 }

de.srsoftware.oidc.datastore.file/src/main/java/de/srsoftware/oidc/datastore/file/FileStore.java

@@ -1,8 +1,10 @@
 /* © SRSoftware 2024 */
 package de.srsoftware.oidc.datastore.file; /* © SRSoftware 2024 */
+import static de.srsoftware.oidc.api.Constants.EXPIRATION;
 import static de.srsoftware.oidc.api.User.*;
 import static de.srsoftware.utils.Optionals.nullable;
 import static de.srsoftware.utils.Strings.uuid;
+import static java.lang.System.Logger.Level.WARNING;
 import static java.util.Optional.empty;
 
 import de.srsoftware.oidc.api.*;
@@ -17,17 +19,17 @@ import java.time.temporal.ChronoUnit;
 import java.util.*;
 import org.json.JSONObject;
 
-public class FileStore implements AuthorizationService, ClientService, SessionService, UserService {
+public class FileStore implements ClaimAuthorizationService, ClientService, SessionService, UserService {
-	private static final String AUTHORIZATIONS = "authorizations";
+	private static final String AUTHORIZATIONS	 = "authorizations";
-	private static final String CLIENTS        = "clients";
+	private static final String CLIENTS	 = "clients";
-	private static final String CODES          = "codes";
+	private static final String CODES	 = "codes";
-	private static final String EXPIRATION     = "expiration";
+	private static final System.Logger LOG	 = System.getLogger(FileStore.class.getSimpleName());
-	private static final String NAME           = "name";
+	private static final String	   NAME	 = "name";
-	private static final String REDIRECT_URIS  = "redirect_uris";
+	private static final String	   REDIRECT_URIS = "redirect_uris";
-	private static final String SECRET         = "secret";
+	private static final String	   SECRET	 = "secret";
-	private static final String SESSIONS       = "sessions";
+	private static final String	   SESSIONS	 = "sessions";
-	private static final String USERS          = "users";
+	private static final String	   USERS	 = "users";
-	private static final String USER           = "user";
+	private static final String	   USER	 = "user";
 
 	private final Path       storageFile;
 	private final JSONObject json;
@@ -35,6 +37,7 @@ public class FileStore implements AuthorizationService, ClientService, SessionSe
 	private Duration	     sessionDuration = Duration.of(10, ChronoUnit.MINUTES);
 	private Map<String, Client>	     clients	     = new HashMap<>();
 	private Map<String, User>	     accessTokens    = new HashMap<>();
+	private Map<String, String>	     authCodes	     = new HashMap<>();
 
 	public FileStore(File storage, PasswordHasher<String> passwordHasher) throws IOException {
 		this.storageFile    = storage.toPath();
@@ -262,80 +265,51 @@ public class FileStore implements AuthorizationService, ClientService, SessionSe
 		return new Client(clientId, clientData.getString(NAME), clientData.getString(SECRET), redirectUris);
 	}
 
-
+	/*** ClaimAuthorizationService methods ***/
-	/*** Authorization service methods ***/
-
-	@Override
-	public Optional<Authorization> forCode(String code) {
-		var authorizations = json.getJSONObject(AUTHORIZATIONS);
-		if (!authorizations.has(code)) return empty();
-		String authId = authorizations.getString(code);
-		if (!authorizations.has(authId)) {
-			authorizations.remove(code);
-			return empty();
-		}
-		try {
-			var expiration = Instant.ofEpochSecond(authorizations.getLong(authId));
-			if (expiration.isAfter(Instant.now())) {
-				String[] parts = authId.split("@");
-				return Optional.of(new Authorization(parts[1], parts[0], expiration));
-			}
-			authorizations.remove(authId);
-		} catch (Exception ignored) {
-		}
-
-		return empty();
-	}
 
 	@Override
-	public AuthorizationService addCode(Client client, User user, String code) {
+	public ClaimAuthorizationService authorize(User user, Client client, Collection<String> scopes, Instant expiration) {
+		LOG.log(WARNING, "{0}.authorize({1}, {2}, {3}, {4}) not implemented", getClass().getSimpleName(), user.realName(), client.name(), scopes, expiration);
 		var authorizations = json.getJSONObject(AUTHORIZATIONS);
-		authorizations.put(code, authorizationId(user, client));
+		if (!authorizations.has(user.uuid())) authorizations.put(user.uuid(), new JSONObject());
+		var userAuthorizations = authorizations.getJSONObject(user.uuid());
+		if (!userAuthorizations.has(client.id())) userAuthorizations.put(client.id(), new JSONObject());
+		var clientScopes = userAuthorizations.getJSONObject(client.id());
+		for (var scope : scopes) clientScopes.put(scope, expiration.getEpochSecond());
 		save();
 		return this;
 	}
 
-	@Override
+	private AuthResult unauthorized(Collection<String> scopes) {
-	public AuthorizationService authorize(Client client, User user, Instant expiration) {
+		return new AuthResult(List.of(), new HashSet<>(scopes), null);
-		var authorizations = json.getJSONObject(AUTHORIZATIONS);
-		authorizations.put(authorizationId(user, client), expiration.getEpochSecond());
-		return this;
-	}
-
-	private String authorizationId(User user, Client client) {
-		return String.join("@", user.uuid(), client.id());
 	}
 
 	@Override
-	public boolean isAuthorized(Client client, User user) {
+	public AuthResult getAuthorization(User user, Client client, Collection<String> scopes) {
-		var authorizations = json.getJSONObject(AUTHORIZATIONS);
+		var authorizations     = json.getJSONObject(AUTHORIZATIONS);
-		var authId	   = authorizationId(user, client);
+		var userAuthorizations = authorizations.has(user.uuid()) ? authorizations.getJSONObject(user.uuid()) : null;
-		if (!authorizations.has(authId)) return false;
+		if (userAuthorizations == null) return unauthorized(scopes);
-
+		var clientScopes = userAuthorizations.has(client.id()) ? userAuthorizations.getJSONObject(client.id()) : null;
-		try {
+		if (clientScopes == null) return unauthorized(scopes);
-			if (Instant.ofEpochSecond(authorizations.getLong(authId)).isAfter(Instant.now())) return true;
+		var now	       = Instant.now();
-		} catch (Exception ignored) {
+		var authorizedScopes   = new ArrayList<AuthorizedScope>();
+		var unauthorizedScopes = new HashSet<String>();
+		for (var scope : scopes) {
+			if (clientScopes.has(scope)) {
+				var expiration = Instant.ofEpochSecond(clientScopes.getLong(scope));
+				if (expiration.isAfter(now)) {
+					authorizedScopes.add(new AuthorizedScope(scope, expiration));
+				} else {
+					unauthorizedScopes.add(scope);
+				}
+			}
 		}
-		revoke(client, user);
+		return new AuthResult(authorizedScopes, unauthorizedScopes, authCode(user, client));
-		return false;
-	}
-
-	@Override
-	public List<User> authorizedUsers(Client client) {
-		return List.of();
 	}
 
-	@Override
+	private String authCode(User user, Client client) {
-	public List<Client> authorizedClients(User user) {
+		var code = uuid();
-		return List.of();
+		authCodes.put(user.uuid() + "@" + client.id(), code);
-	}
+		return code;
-
-	@Override
-	public AuthorizationService revoke(Client client, User user) {
-		var authorizations = json.getJSONObject(AUTHORIZATIONS);
-		var authId	   = authorizationId(user, client);
-		if (!authorizations.has(authId)) return this;
-		authorizations.remove(authId);
-		return save();
 	}
 }

de.srsoftware.oidc.web/src/main/resources/en/authorization.html

@@ -10,14 +10,18 @@
 <body>
     <nav></nav>
     <div id="content" style="display: none">
-        <h1>Authorization</h1>
+        A relying party, <span id="rp">unknown</span>, requested access to the following information:
-        Confirmation required: are you shure you want to grant access to <span id="name">some client</span>?
+        <ul id="scopes">
-        <button type="button" onclick="grantAutorization(1)">Yes - 1 day</button>
+
-        <button type="button" onclick="grantAutorization(7)">Yes - 1 week</button>
+        </ul>
-        <button type="button" onclick="grantAutorization(30)">Yes - 1 month</button>
+        Do you consent to share this information with <span id="rp2">unknown</span>?
-        <button type="button" onclick="grantAutorization(365)">Yes - 1 year</button>
+        <button type="button" onclick="grantAutorization(1)">Yes - for 1 day</button>
+        <button type="button" onclick="grantAutorization(7)">Yes - for 1 week</button>
+        <button type="button" onclick="grantAutorization(30)">Yes - for 1 month</button>
+        <button type="button" onclick="grantAutorization(365)">Yes - for 1 year</button>
         <button type="button" onclick="denyAutorization()">No</button>
     </div>
     <div id="error" class="error" style="display: none"></div>
+    <div id="missing_scopes" class="error" style="display: none">Authorization response contained neither list of <em>unauthorized scopes</em> nor list of <em>authorized scopes</em>! This is a server problem.</div>
 </body>
 </html>

de.srsoftware.oidc.web/src/main/resources/en/authorization.js

@@ -1,32 +1,57 @@
 var params = new URLSearchParams(window.location.search)
 var json = Object.fromEntries(params);
+var scopes = {};
 
 function showConfirmationDialog(name){
     get('name').innerHTML = name;
     show('content');
 }
 
+async function showScope(response,scope){
+    if (response.ok){
+        var content = await response.text();
+        get('scopes').innerHTML += content;
+    } else {
+        get('scopes').innerHTML += '<li>'+scope+' (???)</li>';
+    }
+}
+
 async function handleResponse(response){
     if (response.ok){
         var json = await response.json();
-        console.log("handleResponse(ok) ←",json);
+        if (json.rp) {
-        if (!json.confirmed){
+            setText("rp",json.rp);
-            showConfirmationDialog(json.name);
+            setText("rp2",json.rp);
-        } else {
-        console.log('redirecting to '+json.redirect_uri+'?code='+json.code+'&state='+json.state+'&scope=openid');
-            redirect(json.redirect_uri+'?code='+json.code+'&state='+json.state+'&scope=openid');
         }
-        return;
+        get('scopes').innerHTML = '';
+        if (json.unauthorized_scopes){
+            scopes = json.unauthorized_scopes;
+            for (var scope of json.unauthorized_scopes){
+                fetch(web+"scopes/"+scope+".html").then(response => showScope(response,scope))
+            }
+            show("content");
+            return;
+        }
+        if (json.scope){
+            var url = params.get('redirect_uri') + '?' + new URLSearchParams(json).toString();
+            redirect(url);
+            return;
+        }
+        show('missing_scopes');
     } else {
-        var json = await response.json();
+        console.log(response);
-        console.log("handleResponse(error) ←",json);
+        if (response.status == 401){
-        get('error').innerHTML = "Error: <br/>"+JSON.stringify(json);
+            login();
+            return;
+        }
+        var text = await response.text();
+        setText('error',"Error: <br/>"+text);
         show('error');
     }
 }
 
 function grantAutorization(days){
-    json.days = days;
+    json['authorized'] = { days : days, scopes : scopes};
     backendAutorization();
 }
 

de.srsoftware.oidc.web/src/main/resources/en/common.js

@@ -25,6 +25,10 @@ function hide(id){
     get(id).style.display = 'none';
 }
 
+function login(){
+    redirect('login.html?return_to='+encodeURIComponent(window.location.href));
+}
+
 function redirect(page){
     window.location.href = page;
 }

de.srsoftware.oidc.web/src/main/resources/en/scopes/email.html

@@ -0,0 +1 @@
+<li>Your email address</li>

de.srsoftware.oidc.web/src/main/resources/en/scopes/openid.html

@@ -0,0 +1 @@
+<li>Your OpenID – let the relying party know who you are</li>

de.srsoftware.oidc.web/src/main/resources/en/style.css

@@ -33,3 +33,7 @@ form th{
 .warning{
     color: yellow;
 }
+
+.error{
+    background-color: red;
+}

de.srsoftware.oidc.web/src/main/resources/en/user.js

@@ -1,7 +1,7 @@
 var user = null;
 async function handleUser(response){
     if (response.status == UNAUTHORIZED) {
-        redirect('login.html?return_to='+encodeURIComponent(window.location.href));
+        login();
         return;
     }
     if (response.ok){