This repository has been archived by the owner on Aug 21, 2024. It is now read-only.
Fixed a bug with minikube client local file loading. #5482
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
HexaField
suggested changes
Mar 15, 2022
packages/server-core/src/media/storageprovider/getCachedAsset.ts
Outdated
Show resolved
Hide resolved
getCachedAsset.ts and scene-parser.ts:parseSceneDataCacheURLS always use cacheDomain for generating URLs. On minikube, though, when using local storage, the client and server need different addresses. The server needs to use host.minikube.internal to punch out of minikube to the host machine's localhost, while the client just needs to use localhost (or whatever the local IP is). New logic passes the variable 'internal' around, which is initially sourced from whether the instigating API call came internally (params.provider == null). If true, then it'll use host.minikube.interal:<localStorageProviderPort>, or just host.minikube.internal if localStorageProviderPort isn't defined. Fixed a bug in adminLocations fetching that rebrands location_setting to locationSetting, as we do with non-admin locations.
barankyle
force-pushed
the
minikube-url-fix
branch
from
a672d66
to
7cadaef
Compare
lalalune
referenced
this pull request
in TheNexusCity/XREngine
Mar 20, 2022
* parse gltf ecs data before applying matrix auto update, gltf ecs entities use bitecs transform store * random tsc error fix * Add sorting to admin panel tables (#5461) * implemet logout for admin system * implement logout on admin dashboard * implement filter for user table * add sorting to tables * add user role filter * fix type error * fix project sorting * fix icon style * fix admin nav icon * fix /admin/location alter message * fix drawer style * implemet logout for admin system * implement logout on admin dashboard * implement filter for user table * add sorting to tables * add user role filter * fix type error * fix project sorting * fix icon style * fix admin nav icon * fix /admin/location alter message * fix drawer style * center align column headings in project route of admin panel * refactoring remove unwanted code * refactor admin code * fix update location issue Co-authored-by: Zulqarnain Hanif <30355034+zulqarnainhanif@users.noreply.github.com> * Location type not pre-populating in edit modal (Fix) (#5494) * fix update for location * fix type error * Changes to initially load gameserver with location (#5231) * Changes to initially load gameserver with location * Temporary changed to current branch * QAT triggered * Changed branch name * Changes to add gameserver-load service * Added check to verify instance id * Reverted github QAT action * Updated UI to handle service response * Cleanup * Added logging to check for ip address * Added logging * Added payload to service * Updated variables returned * Added checks for same ip and added scene id * Changed .value from use effect dependency * Added spawn location check based on gameserver name. * Changed to k8 sdk call * Removed logging * Changed pipeline to qat * Added translations * Fixed formatting * Removed kubernetes object check * Graph changes on admin panel (#5471) * update graph on admin panel * fix on change function * update services and fetching logic * add dependencies and update date adapter * use mobile view since desktop view is creating some issue * add custom solution for date time range selection * remove extra code * State sync for physics (#5480) * Generate debug physics cube on key press. (For testing state sync stuff) * Dispatch spawnDebugPhysicsObject against key event. * Sync linear velocity over the network and use it for when updating physics state of remote bodies. * Update debug physics object generation function to work relative to new model functions logic. * Rename the velocity member of VelocityComponent to linearVelocity as it represents the linear velocity of the object. * Add angular velocity to VelocityComponent. And sync it across the network. * Remove extra logs. * Revert "Remove extra logs." This reverts commit 9c26e8d. * Remove extra logs. * Move debug cube binding to AvatarInputSchema. * Move physics debug toggle to AvatarInputSchema key binding. * Revert "Auxiliary commit to revert individual files from d5cdb53" This reverts commit ecf5358d6e0ffada455e331506204f132e4e3aaa. * Rename VelocityComponent fields. * Refactor some code. * Add isDev to check around debug key bindings. * Run prettier. * make physics debug keybind a toggle Co-authored-by: Josh Field <joshfield999@gmail.com> * replace scroll view cursor with should write arg * fix test * fix edge case crash * Issues in admin/avatars (#5501) * fix avatar search * revert package file changes Co-authored-by: Zulqarnain Hanif <30355034+zulqarnainhanif@users.noreply.github.com> * Multi-button Combination Input (#5499) * remove delta from input schema and implement combination input functionality * add tests, fix bugs * Netcode Fix Fixed Tick Bug & No-change Desync (#5511) * ignore component changed periodically, send originating userIndex across network * userIndex per packet instead of per entity * bump simple git * Miscellaneous VR bug fixes (#5512) * fix xr camera layer, controller and rig bugs * fix vr button spam crash * implement feedback * Fixed bug in rendering private project information in upload modal. (#5510) * Improved transport reconnection logic. (#5515) Made a lot of event listeners use named functions so that they can be removed on disconnect. Added *_RECONNECTED message after re-creation of transports. Saving current displayed gameserver warnings in local state, only clearing if update matches the currently displayed error, e.g. CHANNEL_RECONNECTED should not clear warning if it's displaying INSTANCE_DISCONNECTED. * Bump ethereal version, removed unused react-lottie * Update Color scheme for Editor (#5504) * update editor theme * change heirarchy node hover color * update editor color scheme * update numeric input stepper color * fix color of various fields and labels * update toolbar styling and improve purple palette * update dock tab color in editor * update styling editor base route * update create project and uninstall dropdown color scheme * update editor setting button color scheme * update editor base route styling * update styling in editor * Fixed a bug with minikube client local file loading. (#5482) getCachedAsset.ts and scene-parser.ts:parseSceneDataCacheURLS always use cacheDomain for generating URLs. On minikube, though, when using local storage, the client and server need different addresses. The server needs to use host.minikube.internal to punch out of minikube to the host machine's localhost, while the client just needs to use localhost (or whatever the local IP is). New logic passes the variable 'internal' around, which is initially sourced from whether the instigating API call came internally (params.provider == null). If true, then it'll use host.minikube.interal:<localStorageProviderPort>, or just host.minikube.internal if localStorageProviderPort isn't defined. Fixed a bug in adminLocations fetching that rebrands location_setting to locationSetting, as we do with non-admin locations. * Update Admin color scheme (#5516) * update editor theme * change heirarchy node hover color * update editor color scheme * update numeric input stepper color * fix color of various fields and labels * update toolbar styling and improve purple palette * update dock tab color in editor * update styling editor base route * update create project and uninstall dropdown color scheme * update admin color scheme * update editor setting button color scheme * update drawer component to use scss * update editor base route styling * update styling in editor * update admin dashboard color scheme * fix styling of editor create model * fix project admin route background color * update table search button color scheme in different admin routes remove extra styling * update bots admin page * update settings admin page color scheme * Clean user login (#5472) * redesigned confirm email and showed notification after email have sent * added loading while sending sms or email * added i18n Co-authored-by: Josh Field <joshfield999@gmail.com> * Added cluster autoscaler installation to AWS setup instructions. (#5527) * Add new inventory changes * Fix missing storage key * inventory view prototype * inventory slot drag & drop feature * Merge inventory * implemet logout for admin system * implement logout on admin dashboard * implement filter for user table * add sorting to tables * add user role filter * update graph on admin panel * fix on change function * update services and fetching logic * add dependencies and update date adapter * redesigned confirm email and showed notification after email have sent * added loading while sending sms or email * Dispatch spawnDebugPhysicsObject against key event. * Sync linear velocity over the network and use it for when updating physics state of remote bodies. * Update debug physics object generation function to work relative to new model functions logic. * Rename the velocity member of VelocityComponent to linearVelocity as it represents the linear velocity of the object. * Add angular velocity to VelocityComponent. And sync it across the network. * Remove extra logs. * Revert "Remove extra logs." This reverts commit 9c26e8d. * Remove extra logs. * Fixed a bug with minikube client local file loading. getCachedAsset.ts always uses cacheDomain for generating URLs. On minikube, though, when using local storage, the client and server need different addresses. The server needs to use 10.0.2.2 to punch out of minikube to the host machine's localhost, while the client just needs to use localhost. New logic assumes that localStorageProvider will be 10.0.2.2, and so alterations to that are done if new parameter 'clientFetch' is explicitly true (defaults to false). scene.class.ts now accepts query parameter clientFetch and passes it on. Fixed a bug in adminLocations fetching that rebrands location_setting to locationSettings, as we do with non-admin locations. * refactored buttons and added auto hide with animation * improve buttons animation and appearance and checked its functionalities * enabled all buttons by default and put facetrack to work * fixed animation issue * fix rotation * Changes to initially load gameserver with location * Temporary changed to current branch * QAT triggered * Changed branch name * Changes to add gameserver-load service * Added check to verify instance id * Reverted github QAT action * Updated UI to handle service response * Cleanup * Added logging to check for ip address * Added logging * Added payload to service * Updated variables returned * Added checks for same ip and added scene id * Changed .value from use effect dependency * Added spawn location check based on gameserver name. * Changed to k8 sdk call * Removed logging * Changed pipeline to qat * Added translations * Fixed formatting * Removed kubernetes object check * Bring nexus dev up to date, remove inventory from dev, prep to move to project * Smol patches Co-authored-by: Josh Field <joshfield999@gmail.com> Co-authored-by: Hamza Mushtaq <hamzamushtaq34@hotmail.com> Co-authored-by: kimenyikevin <52419102+kimenyikevin@users.noreply.github.com> Co-authored-by: Zulqarnain Hanif <30355034+zulqarnainhanif@users.noreply.github.com> Co-authored-by: Hanzla Mateen <hanzlamateen@live.com> Co-authored-by: Kyle Baran <kbaran@bitscoop.com> Co-authored-by: Gheric Speiginer <gheric.speiginer@gmail.com> Co-authored-by: IRANKUNDA Fabrice <58092199+FabriceIRANKUNDA@users.noreply.github.com> Co-authored-by: dreamworker <metadev26@gmail.com> Co-authored-by: kimenyi kevin <kimenyikevin@gmail.com> Co-authored-by: FabriceIRANKUNDA <irankundafabrice8@gmail.com>
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
getCachedAsset.ts always uses cacheDomain for generating URLs. On minikube, though, when using local storage,
the client and server need different addresses. The server needs to use 10.0.2.2 to punch out of minikube to
the host machine's localhost, while the client just needs to use localhost.
New logic assumes that localStorageProvider will be 10.0.2.2, and so alterations to that are done if new parameter
'clientFetch' is explicitly true (defaults to false). scene.class.ts now accepts query parameter clientFetch and passes
it on.
Fixed a bug in adminLocations fetching that rebrands location_setting to locationSettings, as we do with non-admin locations.
Checklist
npm run checknpm run lintnpm run test:packagesnpm run build-clientReferences
References to pertaining issue(s)
QA Steps
git checkout pr_branch_namenpm installnpm run dev-reinitnpm run devList any additional steps required to QA the changes of this PR, as well as any supplemental images or videos.
Reviewers
Reviewers for this PR