wwu-webtech / descriptive-alt-text-generator

MIT License
0 stars 0 forks source link

Authentication mechanism #5

Closed alexkefer closed 6 months ago

alexkefer commented 7 months ago

Added the authentication module and testing it to connect to ContactUs if the authentication fails just to see how it all works to begin with, if works I already separated the portion we want hidden into a separate component to simplify the authorization process. This is a test PR. DO NOT MERGE!

alexkefer commented 7 months ago

Got it working to a point, I sent an email explaining the current situation I am at with it.

WWU-MaxBronsema commented 7 months ago

Slick. Mainly the config file got the additional routing information. It seems like the other commits are related to moving the text generator into it's own file. Hopefully the new token enabling makes this work.

alexkefer commented 6 months ago

Currently hides the homepage to not logged in users, reroutes them to the login page, then once logged in, routes them back to the working web app. Next is working on getting user data from the page in order to continue developing the database based on the user logged in on the personal statistics page. I need the account information in order to craft the database properly.

Before MERGE to main we will also need to add the main address to the Redirect URI as the address to the page will change once this is on main. otherwise this will not work.

davidengebretson commented 6 months ago

You want a review and not a merge?

From: alex kefer @.> Sent: Monday, April 15, 2024 2:57 PM To: wwu-webtech/descriptive-alt-text-generator @.> Cc: David Engebretson @.>; Review requested @.> Subject: Re: [wwu-webtech/descriptive-alt-text-generator] Authentication mechanism (PR #5)

@alexkeferhttps://github.com/alexkefer requested your review on: #5https://github.com/wwu-webtech/descriptive-alt-text-generator/pull/5 Authentication mechanism.

- Reply to this email directly, view it on GitHubhttps://github.com/wwu-webtech/descriptive-alt-text-generator/pull/5#event-12478183938, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKX5QZRFQWOHUX7Z5GWNBLDY5REJ7AVCNFSM6AAAAABF2CJRN2VHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJSGQ3TQMJYGM4TGOA. You are receiving this because your review was requested.Message ID: @.**@.>>

davidengebretson commented 6 months ago

Which account information are you referencing In the below sentence: "I need the account information in order to craft the database properly."?

From: alex kefer @.> Sent: Monday, April 15, 2024 2:57 PM To: wwu-webtech/descriptive-alt-text-generator @.> Cc: David Engebretson @.>; Review requested @.> Subject: Re: [wwu-webtech/descriptive-alt-text-generator] Authentication mechanism (PR #5)

Currently hides the homepage to not logged in users, reroutes them to the login page, then once logged in, routes them back to the working web app. Next is working on getting user data from the page in order to continue developing the database based on the user logged in on the personal statistics page. I need the account information in order to craft the database properly.

- Reply to this email directly, view it on GitHubhttps://github.com/wwu-webtech/descriptive-alt-text-generator/pull/5#issuecomment-2057878718, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKX5QZSIWGN5VXCFHRWMLKLY5REJPAVCNFSM6AAAAABF2CJRN2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJXHA3TQNZRHA. You are receiving this because your review was requested.Message ID: @.**@.>>

alexkefer commented 6 months ago

Hey David, I am still working on it though I added you as a reviewer so you can see its progress in updates and so you can verify the login works on your end and is accessible and understandable to use (It should be as it is simply running you through the western portal as most other western applications do). Technically it does work as a login mechanism but I am continuing to iterate in order to prepare the personal data pages and administration pages. So whilst it is at a point where with a quick update to the Redirect URI it can be merged, I think with a couple more days I can have it ready to be merged with feature sets ready for future iterations.

On Mon, Apr 15, 2024 at 3:10 PM davidengebretson @.***> wrote:

Which account information are you referencing In the below sentence: "I need the account information in order to craft the database properly."?

From: alex kefer @.> Sent: Monday, April 15, 2024 2:57 PM To: wwu-webtech/descriptive-alt-text-generator @.> Cc: David Engebretson @.>; Review requested @.> Subject: Re: [wwu-webtech/descriptive-alt-text-generator] Authentication mechanism (PR #5)

Currently hides the homepage to not logged in users, reroutes them to the login page, then once logged in, routes them back to the working web app. Next is working on getting user data from the page in order to continue developing the database based on the user logged in on the personal statistics page. I need the account information in order to craft the database properly.

- Reply to this email directly, view it on GitHub< https://github.com/wwu-webtech/descriptive-alt-text-generator/pull/5#issuecomment-2057878718>, or unsubscribe< https://github.com/notifications/unsubscribe-auth/AKX5QZSIWGN5VXCFHRWMLKLY5REJPAVCNFSM6AAAAABF2CJRN2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJXHA3TQNZRHA

. You are receiving this because your review was requested.Message ID: @.**@.>>

— Reply to this email directly, view it on GitHub https://github.com/wwu-webtech/descriptive-alt-text-generator/pull/5#issuecomment-2057897571, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQPPYB7TEHWI7OKBVYKWDYDY5RF7BAVCNFSM6AAAAABF2CJRN2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJXHA4TONJXGE . You are receiving this because you were mentioned.Message ID: @.***>

davidengebretson commented 6 months ago

Are you taking care of the Redirect URI, or is there something I can do to help with that?

From: alex kefer @.> Sent: Monday, April 15, 2024 3:16 PM To: wwu-webtech/descriptive-alt-text-generator @.> Cc: David Engebretson @.>; Review requested @.> Subject: Re: [wwu-webtech/descriptive-alt-text-generator] Authentication mechanism (PR #5)

Hey David, I am still working on it though I added you as a reviewer so you can see its progress in updates and so you can verify the login works on your end and is accessible and understandable to use (It should be as it is simply running you through the western portal as most other western applications do). Technically it does work as a login mechanism but I am continuing to iterate in order to prepare the personal data pages and administration pages. So whilst it is at a point where with a quick update to the Redirect URI it can be merged, I think with a couple more days I can have it ready to be merged with feature sets ready for future iterations.

On Mon, Apr 15, 2024 at 3:10 PM davidengebretson @.<mailto:@.>> wrote:

Which account information are you referencing In the below sentence: "I need the account information in order to craft the database properly."?

From: alex kefer @.<mailto:@.>> Sent: Monday, April 15, 2024 2:57 PM To: wwu-webtech/descriptive-alt-text-generator @.<mailto:@.>> Cc: David Engebretson @.<mailto:@.>>; Review requested @.<mailto:@.>> Subject: Re: [wwu-webtech/descriptive-alt-text-generator] Authentication mechanism (PR #5)

Currently hides the homepage to not logged in users, reroutes them to the login page, then once logged in, routes them back to the working web app. Next is working on getting user data from the page in order to continue developing the database based on the user logged in on the personal statistics page. I need the account information in order to craft the database properly.

- Reply to this email directly, view it on GitHub< https://github.com/wwu-webtech/descriptive-alt-text-generator/pull/5#issuecomment-2057878718>, or unsubscribe< https://github.com/notifications/unsubscribe-auth/AKX5QZSIWGN5VXCFHRWMLKLY5REJPAVCNFSM6AAAAABF2CJRN2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJXHA3TQNZRHA

. You are receiving this because your review was requested.Message ID: @.**@.mailto:***@***.******@***.***>>

— Reply to this email directly, view it on GitHub https://github.com/wwu-webtech/descriptive-alt-text-generator/pull/5#issuecomment-2057897571, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQPPYB7TEHWI7OKBVYKWDYDY5RF7BAVCNFSM6AAAAABF2CJRN2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJXHA4TONJXGE . You are receiving this because you were mentioned.Message ID: @.<mailto:@.>>

— Reply to this email directly, view it on GitHubhttps://github.com/wwu-webtech/descriptive-alt-text-generator/pull/5#issuecomment-2057903116, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKX5QZQNSK7H6ZSGXRVPS53Y5RGRHAVCNFSM6AAAAABF2CJRN2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJXHEYDGMJRGY. You are receiving this because your review was requested.Message ID: @.**@.>>

alexkefer commented 6 months ago

I was going to put a ticket in after my class to have that added to the redirect uri. Basically just the link needs to be the same format but the base (pre slashes) needs to be updated to the non branch link. If you want to get that in you are welcome to do so. Alex


From: davidengebretson @.> Sent: Monday, April 15, 2024 3:37:33 PM To: wwu-webtech/descriptive-alt-text-generator @.> Cc: alex kefer @.>; Mention @.> Subject: Re: [wwu-webtech/descriptive-alt-text-generator] Authentication mechanism (PR #5)

Are you taking care of the Redirect URI, or is there something I can do to help with that?

From: alex kefer @.> Sent: Monday, April 15, 2024 3:16 PM To: wwu-webtech/descriptive-alt-text-generator @.> Cc: David Engebretson @.>; Review requested @.> Subject: Re: [wwu-webtech/descriptive-alt-text-generator] Authentication mechanism (PR #5)

Hey David, I am still working on it though I added you as a reviewer so you can see its progress in updates and so you can verify the login works on your end and is accessible and understandable to use (It should be as it is simply running you through the western portal as most other western applications do). Technically it does work as a login mechanism but I am continuing to iterate in order to prepare the personal data pages and administration pages. So whilst it is at a point where with a quick update to the Redirect URI it can be merged, I think with a couple more days I can have it ready to be merged with feature sets ready for future iterations.

On Mon, Apr 15, 2024 at 3:10 PM davidengebretson @.<mailto:@.>> wrote:

Which account information are you referencing In the below sentence: "I need the account information in order to craft the database properly."?

From: alex kefer @.<mailto:@.>> Sent: Monday, April 15, 2024 2:57 PM To: wwu-webtech/descriptive-alt-text-generator @.<mailto:@.>> Cc: David Engebretson @.<mailto:@.>>; Review requested @.<mailto:@.>> Subject: Re: [wwu-webtech/descriptive-alt-text-generator] Authentication mechanism (PR #5)

Currently hides the homepage to not logged in users, reroutes them to the login page, then once logged in, routes them back to the working web app. Next is working on getting user data from the page in order to continue developing the database based on the user logged in on the personal statistics page. I need the account information in order to craft the database properly.

- Reply to this email directly, view it on GitHub< https://github.com/wwu-webtech/descriptive-alt-text-generator/pull/5#issuecomment-2057878718>, or unsubscribe< https://github.com/notifications/unsubscribe-auth/AKX5QZSIWGN5VXCFHRWMLKLY5REJPAVCNFSM6AAAAABF2CJRN2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJXHA3TQNZRHA

. You are receiving this because your review was requested.Message ID: @.**@.mailto:***@***.******@***.***>>

— Reply to this email directly, view it on GitHub https://github.com/wwu-webtech/descriptive-alt-text-generator/pull/5#issuecomment-2057897571, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQPPYB7TEHWI7OKBVYKWDYDY5RF7BAVCNFSM6AAAAABF2CJRN2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJXHA4TONJXGE . You are receiving this because you were mentioned.Message ID: @.<mailto:@.>>

— Reply to this email directly, view it on GitHubhttps://github.com/wwu-webtech/descriptive-alt-text-generator/pull/5#issuecomment-2057903116, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKX5QZQNSK7H6ZSGXRVPS53Y5RGRHAVCNFSM6AAAAABF2CJRN2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJXHEYDGMJRGY. You are receiving this because your review was requested.Message ID: @.**@.>>

— Reply to this email directly, view it on GitHubhttps://github.com/wwu-webtech/descriptive-alt-text-generator/pull/5#issuecomment-2057928639, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPPYBZDSPJ4THWVEJ2LUZDY5RJC3AVCNFSM6AAAAABF2CJRN2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJXHEZDQNRTHE. You are receiving this because you were mentioned.Message ID: @.***>

davidengebretson commented 6 months ago

I’ll let you take care of that. 😊

From: alex kefer @.> Sent: Monday, April 15, 2024 3:56 PM To: wwu-webtech/descriptive-alt-text-generator @.> Cc: David Engebretson @.>; Review requested @.> Subject: Re: [wwu-webtech/descriptive-alt-text-generator] Authentication mechanism (PR #5)

I was going to put a ticket in after my class to have that added to the redirect uri. Basically just the link needs to be the same format but the base (pre slashes) needs to be updated to the non branch link. If you want to get that in you are welcome to do so. Alex


From: davidengebretson @.<mailto:@.>> Sent: Monday, April 15, 2024 3:37:33 PM To: wwu-webtech/descriptive-alt-text-generator @.<mailto:@.>> Cc: alex kefer @.<mailto:@.>>; Mention @.<mailto:@.>> Subject: Re: [wwu-webtech/descriptive-alt-text-generator] Authentication mechanism (PR #5)

Are you taking care of the Redirect URI, or is there something I can do to help with that?

From: alex kefer @.<mailto:@.>> Sent: Monday, April 15, 2024 3:16 PM To: wwu-webtech/descriptive-alt-text-generator @.<mailto:@.>> Cc: David Engebretson @.<mailto:@.>>; Review requested @.<mailto:@.>> Subject: Re: [wwu-webtech/descriptive-alt-text-generator] Authentication mechanism (PR #5)

Hey David, I am still working on it though I added you as a reviewer so you can see its progress in updates and so you can verify the login works on your end and is accessible and understandable to use (It should be as it is simply running you through the western portal as most other western applications do). Technically it does work as a login mechanism but I am continuing to iterate in order to prepare the personal data pages and administration pages. So whilst it is at a point where with a quick update to the Redirect URI it can be merged, I think with a couple more days I can have it ready to be merged with feature sets ready for future iterations.

On Mon, Apr 15, 2024 at 3:10 PM davidengebretson @.<mailto:@.mailto:***@***.***%3cmailto:***@***.***>> wrote:

Which account information are you referencing In the below sentence: "I need the account information in order to craft the database properly."?

From: alex kefer @.<mailto:@.mailto:***@***.***%3cmailto:***@***.***>> Sent: Monday, April 15, 2024 2:57 PM To: wwu-webtech/descriptive-alt-text-generator @.<mailto:@.mailto:***@***.***%3cmailto:***@***.***>> Cc: David Engebretson @.<mailto:@.mailto:***@***.***%3cmailto:***@***.***>>; Review requested @.<mailto:@.mailto:***@***.***%3cmailto:***@***.***>> Subject: Re: [wwu-webtech/descriptive-alt-text-generator] Authentication mechanism (PR #5)

Currently hides the homepage to not logged in users, reroutes them to the login page, then once logged in, routes them back to the working web app. Next is working on getting user data from the page in order to continue developing the database based on the user logged in on the personal statistics page. I need the account information in order to craft the database properly.

- Reply to this email directly, view it on GitHub< https://github.com/wwu-webtech/descriptive-alt-text-generator/pull/5#issuecomment-2057878718>, or unsubscribe< https://github.com/notifications/unsubscribe-auth/AKX5QZSIWGN5VXCFHRWMLKLY5REJPAVCNFSM6AAAAABF2CJRN2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJXHA3TQNZRHA

. You are receiving this because your review was requested.Message ID: @.**@.mailto:***@***.******@***.***<mailto:***@***.******@***.***%3cmailto:***@***.******@***.***>>>

— Reply to this email directly, view it on GitHub https://github.com/wwu-webtech/descriptive-alt-text-generator/pull/5#issuecomment-2057897571, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQPPYB7TEHWI7OKBVYKWDYDY5RF7BAVCNFSM6AAAAABF2CJRN2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJXHA4TONJXGE . You are receiving this because you were mentioned.Message ID: @.<mailto:@.mailto:***@***.***%3cmailto:***@***.***>>

— Reply to this email directly, view it on GitHubhttps://github.com/wwu-webtech/descriptive-alt-text-generator/pull/5#issuecomment-2057903116, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKX5QZQNSK7H6ZSGXRVPS53Y5RGRHAVCNFSM6AAAAABF2CJRN2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJXHEYDGMJRGY. You are receiving this because your review was requested.Message ID: @.**@.mailto:***@***.******@***.***>>

— Reply to this email directly, view it on GitHubhttps://github.com/wwu-webtech/descriptive-alt-text-generator/pull/5#issuecomment-2057928639, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AQPPYBZDSPJ4THWVEJ2LUZDY5RJC3AVCNFSM6AAAAABF2CJRN2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJXHEZDQNRTHE. You are receiving this because you were mentioned.Message ID: @.<mailto:@.>>

— Reply to this email directly, view it on GitHubhttps://github.com/wwu-webtech/descriptive-alt-text-generator/pull/5#issuecomment-2057946719, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AKX5QZRNVRVCN6TKCAAAO7LY5RLHTAVCNFSM6AAAAABF2CJRN2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANJXHE2DMNZRHE. You are receiving this because your review was requested.Message ID: @.**@.>>