Add LDAP support

please add LDAP support for connecting to Windows Active Directory.
https://github.com/Nordes/IdentityServer4.LdapExtension
thanks a lot ;)

Hi @omidghorbani
thanks for tip, I will check it and consider add to road map.

Hey @skoruba any news on that? I've seen a branch named windows-auth-v2, is it in progress? Just wondering if I could fork from that branch or I should extend this implementation for my own requirements (Active Directory behind IIS) + OpenID Connect + ASPNET Identity base for external users.
Anyway great job, I am impressed by your work and really enjoy this project. I'd like to contribute in the future :)

Hi, i am looking for Active Directory integration too.

Any updates on LDAP support


:label: :bug: corrected type definition

the type definition for userFound and silentRenewError were missing parameters.

Merged, thanks!

And published in version 3.1.4


PDF downloads

See plugin here https://github.com/meff34/docsify-to-pdf-converter
Also docsify awesome https://github.com/docsifyjs/awesome-docsify

Adding it now

Adding it now

https://github.com/NebraLtd/Helium-Guides/blob/main/pdf/readme.pdf

https://github.com/NebraLtd/Helium-Guides/blob/main/pdf/readme.pdf

@ryanteck how do we generate them for other pages? And does this auto-generate somehow or we can build into a GitHub workflow?

@ryanteck how do we generate them for other pages? And does this auto-generate somehow or we can build into a GitHub workflow?

From here - https://github.com/GoogleChrome/puppeteer/blob/master/docs/api.md#pagepdfoptions
And here - https://github.com/GoogleChrome/puppeteer/blob/master/docs/api.md#pageemulatemediamediatype
It looks like there is quite a lot of customisation options, including separate print templates etc.
Wonder if we can have pdfs of just certain pages / folders from the repo

From here - https://github.com/GoogleChrome/puppeteer/blob/master/docs/api.md#pagepdfoptions
And here - https://github.com/GoogleChrome/puppeteer/blob/master/docs/api.md#pageemulatemediamediatype
It looks like there is quite a lot of customisation options, including separate print templates etc.
Wonder if we can have pdfs of just certain pages / folders from the repo

@ryanteck how do we generate them for other pages? And does this auto-generate somehow or we can build into a GitHub workflow?

Currently for testing I've been running a command locally.
I can likely set it up on github actions or a cron that runs on the Pi that hosts it.

@ryanteck how do we generate them for other pages? And does this auto-generate somehow or we can build into a GitHub workflow?

Currently for testing I've been running a command locally.
I can likely set it up on github actions or a cron that runs on the Pi that hosts it.

PDF generation files because of removing the # from what I can tell.
https://github.com/meff34/docsify-to-pdf-converter/blob/aae1db48cce15ebfae744bb0a046cf5c68f56f1e/src/render.js#L22

PDF generation files because of removing the # from what I can tell.
https://github.com/meff34/docsify-to-pdf-converter/blob/aae1db48cce15ebfae744bb0a046cf5c68f56f1e/src/render.js#L22

Nice find @ryanteck
Started an issue https://github.com/meff34/docsify-to-pdf-converter/issues/24
Maybe we can do a pr that checks for the history router and changes the behaviour?

Nice find @ryanteck
Started an issue https://github.com/meff34/docsify-to-pdf-converter/issues/24
Maybe we can do a pr that checks for the history router and changes the behaviour?

It looks like an abandoned project now so don't think it'll get fixed.
I think the easiest thing will be if I just edit the # out and see if that fixes it.

It looks like an abandoned project now so don't think it'll get fixed.
I think the easiest thing will be if I just edit the # out and see if that fixes it.

Yeah for sure. So just fork it and do our own?

Yeah for sure. So just fork it and do our own?

I don't think should need to do that.
The plugin is installed locally on the server so I just need to remove the # from that.
I'll then set it up to automate generation of the PDF each git push too
Just doing it now.

I don't think should need to do that.
The plugin is installed locally on the server so I just need to remove the # from that.
I'll then set it up to automate generation of the PDF each git push too
Just doing it now.

@ryanteck can we just do one PDF from the whole site or we can do multiple different PDFs?

@ryanteck can we just do one PDF from the whole site or we can do multiple different PDFs?

I think with this plug in it is the whole site.
It might be better instead to use a different piece of software to render induvidual PDFs from each markdown.
I'm currently having issues getting the plugin set up on the Pi server because of lack of IPv4 is causing issues with NPM.

I think with this plug in it is the whole site.
It might be better instead to use a different piece of software to render induvidual PDFs from each markdown.
I'm currently having issues getting the plugin set up on the Pi server because of lack of IPv4 is causing issues with NPM.

Yeah, I guess this will do for now and we can improve in the future

Yeah, I guess this will do for now and we can improve in the future

Was still having issues with this plugin.
For now I've configured the server that it will generate PDFs for files listed in https://github.com/NebraLtd/Helium-Guides/blob/main/pdf-gen/gen-pdfs.sh
Currently I've just got it generating the main ones that need to be provided as PDF which are the overview and quick start guides.
Could possibly in the future move it to be where it generates for every file, however I think for most just the main guides are the ones required to be in PDF form.
These are generated every time the repo's main branch is pushed to, and are uploaded to https://helium.nebra.com/pdfs/


Update README.md

Short Description:
Extended the configuration for the nginx server and web client. For beginners, just to help them through all steps.

#1161
Maybe you should move this to the wiki.

:-1: Web folder is not apart of this repo so we only need to reference where it is and they will have to handle howto config
Which has already been done here: https://github.com/PokemonGoF/PokemonGo-Bot/pull/1202


Cannot get ng-token-auth, devise-token-auth and Rails to work for facebook login

I apologize if this question has been answered before. If it is, please point to the solution. But for the life of it I can not figure out how to get an AngularJS client (using ng-token-auth) and Rails 4.2 server (using devise-token-auth) to work with  Facebook for login/signup, etc.  I have googled this topic and read through a lot of suggestions and ideas but NOT a concrete solution. So I finally decided to ask the experts. Here it goes:
THE SYMPTOM:
When I initiate a $auth.authenticate('facebook') call on client, I get the HTTP request from facebook to
the redirect_url on rails server, at http://localhost:3008/api/v1/auth/facebook/callback which map to
'devise_token_auth/omniauth_callbacks#redirect_callbacks' in routes.rb

GET  /api/v1/auth/:provider/callback(.:format) devise_token_auth/omniauth_callbacks#redirect_callbacks

I also have another route defined in when I do 'rake routes'

GET    /v1/auth/:provider/callback(.:format)        devise_token_auth/omniauth_callbacks#omniauth_success

I am  not sure why I have two routes defined, but only the first one (redirect_callbacks) gets called.
snippet from config/routes.rb
  mount Api::Engine, at: '/api'

    devise_for :users

    # token auth routes available at /api/v1/auth
     namespace :v1, defaults: { format: :json } do
       mount_devise_token_auth_for 'User', at: 'auth'
       resources :users, only: [:index, :show, :create, :update, :destroy]
     end

Snippet from config/devise_token_auth.rb
  # This route will be the prefix for all oauth2 redirect callbacks. For
  # example, using the default '/omniauth', the github oauth2 provider will
  # redirect successful authentications to '/omniauth/github/callback'
  #config.omniauth_prefix = "/omniauth"

  config.omniauth_prefix = "/api/v1/auth"

The Application Controller:
module Api
  module V1
    class ApplicationController < ActionController::Base
      # required by devise_token_auth
      include DeviseTokenAuth::Concerns::SetUserByToken
      respond_to :json

      before_filter :authenticate_user!, except: [:new, :create]

      # protect_from_forgery with: :exception
      # Prevent CSRF attacks by using :null_session
      protect_from_forgery with: :null_session
      before_action :configure_permitted_parameters, if: :devise_controller?

      def set_csrf_cookie_for_ng
        cookies['XSRF-TOKEN'] = form_authenticity_token if protect_against_forgery?
      end

      def configure_permitted_parameters
        devise_parameter_sanitizer.for(:sign_in) << :session
        devise_parameter_sanitizer.for(:sign_up) << :favorite_color
        devise_parameter_sanitizer.for(:account_update) << :favorite_color
        devise_parameter_sanitizer.for(:account_update) << :nickname
      end

      def verified_request?
        super || valid_authenticity_token?(session, request.headers['X-XSRF-TOKEN'])
      end

    end
  end
end

My request from facbook to server raises an exception "No resource_class found" at line 184 in devise_token_auth/omniauth_controller_callback.rb:
Note: If I am logged into Facebook from browser, I get the Facebook login page in a new tab. After logging into to Facebook, I get the same exception as above.
    def resource_class(mapping = nil)
      if omniauth_params['resource_class']
        omniauth_params['resource_class'].constantize
      elsif params['resource_class']
        params['resource_class'].constantize
      else
           raise "No resource_class found"
      end
    end

In a debug session the omniauth_params has only following two things:
auth_original_url => http://3000/#/pages/auth/login
omniauth_window_type => sameWindow

As you can see there is no "access_token", "resource_class" or any other fields available in omniauth.params which is causing this exception.  I have checked both client and server configuration multiple times and compared it with documentation for both ng-token-auth and devise-token-auth and it seems to be correct. I somehow can't get this to work. So I am asking for your help.
Following is my client and server configuration if that helps.
Client Configuration:

AngularJS 1.4.8
ng-token-auth (0.0.28)
angular-cookie (4.0.10)
Client runs on http://localhost:3000

Server Configuration

Rails (4.2.4)
devise (3.5.2)
devise_token_auth (0.1.33)
oauth2 (1.1.0)
omniauth-oauth2 (1.4.0)
omniauth-facebook (3.0.0)
Server runs on http://localhost:3008

NOTE: I need devise and devise_token_auth to co-exist to support legacy application.

+1

I have the exact same problem. Were you able to solve it?

I solved this by making the following change
$auth.authenticate(provider)
to
$auth.authenticate(provider, {params: {resource_class: 'User'}}

I managed by overriding omniauth_callbacks_controller and token_validations_controller.
The resource class was missing in controllers so I globally declare

var $resource = @resource
in omniauth_success method of omniauth_callbacks_controller and also use same resource for token_validations_controller .
you can find fully configured app at here
As it is working fine for me.

Sounds like we have a proposed solution here, closing for now.


ARIA1 - Example 1

tested and added link back to github file

Merged - Thanks Cam!  Check your gitignore file though - your .DS_store file came over with this PR.  Not a problem, I'm just going to delete - but just wanted to give you a heads-up for future PRs.


Unable to see Vulnerabilities reported by dependency-check

Hello @jeremylong ,
We have installed the Enterprise Sonarqube server on-prem and pushing project analysis and dependency-check results to this server. But Sonarqube Server is not able to display vulnerabilities that are found by dependency-check. Though we are able to see HTML reports on sonar.
This is happening for any kind of project (MSBuild, Typescript, or Java).
But If I spin up the local Sonarqube server and do the analysis locally with dependency-check it works perfectly fine as expected.
We have also installed Dependency-Check Plugin on the SonarQube server.
SonarQube Version : Version 9.6.1
dependency-check: version 7.2.1
Let me know if you need more information. We are kind of stuck and unable to understand why it is not working on Hosted SonarQube Server

It seems a SonarQube problem,  not a Dependency-Check, I suggest you contact SonarQube

The sonar plugin is here: https://github.com/dependency-check/dependency-check-sonar-plugin
Please raise this issue on the sonar plugin's repo.


Some entity's embedding missing

Some entities return null when I use
pre_trained_kge.get_entity_embeddings(['https://dbpedia.org/resource/Abraham_Lincoln']).
"Abraham Lincoln" is one of a major entity in DBpedia, and it returns null.
It has a DBpedia and Wikipedia page available.
Can you please fix it?
thanks

Hello @umairq ,
I assume that the missing entity issue is related to the example provided in https://github.com/dice-group/dice-embeddings#using-pre-trained-conex-on-dbpedia-03-2022
It might be the case that some entity embeddings are missing. In this document, we elucidated each step taking during preprocessing DBpedia 03-2022.
Cheers

Thanks @Demirrr
Yes I read the documentation.
But I think this entity (i.e., https://dbpedia.org/resource/Abraham_Lincoln' ) does not come under the pre-processing removal criteria. As It is not a literal or any other criteria mentioned in document.

Sorry for the late response.

But I think this entity (i.e., https://dbpedia.org/resource/Abraham_Lincoln' ) does not come under the pre-processing removal criteria. As It is not a literal or any other criteria mentioned in document.
Can you please point out exactly why it is excluded?

Unfortunately I do not have time atm to detect at which step of the preprocessing the mentioned entity is removed.

can you please provide the very basic steps to re-train it on a newer version of DBpedia? (As I have many examples of such missing entities)

We have ContinuousExecute class implemented to continue the training process by using a pre-trained model. Yet, given that you need embedding vectors for entities haven't previously seen, this class would not be helpful. I am afraid, you would need to construct a training dataset containing all triples of your interest and train a model a desired hyper params.
At the time being, DBpedia 2022-03 is the newest version of DBpedia (see https://www.dbpedia.org/blog/dbpedia-snapshot-2022-03-release/)


[WIP] markup events as microdata

Calendar pages contain very simple information that could be presented to the user without them even visiting the page.
At the moment, it seems like search engines don't always understand how to use our calendars to answer the user's query directly and GOV.UK is not always the top result (at least for me).
Here's some examples of google doing the wrong thing (the next bank holiday for England and Wales is in December):

This branch is an attempt to implement this schema: http://schema.org/Event
WIP HTML:

                  <meta itemprop="location" content="England and Wales">

The next bank holiday in England and Wales is

                    <time itemprop="startDate" datetime="2016-12-26">26 December</time>
                    Boxing Day

This doesn't validate according to https://search.google.com/structured-data/testing-tool/u/0/, although the w3c parser didn't have a problem with it.
I'm not sure if this is a sensible way to deal with "location". I've used a meta tag because it feels like the location is implicit: we only refer to the division in the "in" text and this can differ from the location name:
      england-and-wales:      "Cymru a Lloegr"
      england-and-wales_in:   "yng Nghymru a Lloegr"

I think it's not validating because you haven't said what itemtype location is and it defaults to 'Thing' the most generic itemtype, which is okay for something like name, but isn't for location.
https://schema.org/location
The other complaints look more like recommendations more than anything.

@carolinegreen do you have an example that validates? I tried actually adding itemtype attribute to the meta tag but couldn't get that to validate either. 😭
I tried to find a similar example on schema.org but they use a "Thing" as well:

    FedExForum

  Memphis, TN, US
  <meta itemprop="startDate" content="2011-05-20">May 20
  Buy tickets

@MatMoore Yes, I can, by using the extra information provided in microdata example on the page I linked to.  (It's at the bottom.)  Although the actual example doesn't validate in it's original form.  If you take away the meta tag, and nest an address and name within the location as a tag which accepts nested tags it will validate.  Which I realise defeats the point of using the meta tag in this way.
The documentation seems pretty dubious on schema org.  I've read conflicting things from other sources as well.  It needs more thought, and I'm still on 2ndline at the moment.
I'm not suggesting we use this at all, but it is an example that validates.
`

Bank Holiday
England and Wales

The next bank holiday in England and Wales is

26 December
Boxing Day

`

I'm going to close this for now as I haven't had time to look at it in many more detail, and the PR is not mergeable in its current state.


docs: Update "How it works" section in writing REST API documentation.

Updates this section to use the shared api-doc-template.md as a guide, as well as the current documentation for the render-message endpoint. Previously, this section referred to a file that had been removed with the transition to a shared template file.
Fixes #24485.
Notes:

Ideally, this section and the following would be converted to be more like the new feature tutorial, with this section as the general process and the next section following a specific example.

Screenshots and screen captures:

Updated "How it works" section

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

This is great, merged, thanks @laurynmm!


Automations triggers twice

Home Assistant release with the issue:
0.85
Last working Home Assistant release (if known):
Not sure, 0.84.X?
Operating environment (Hass.io/Docker/Windows/etc.):
Windows 10, Python 3.6.8 64-bit
Component/platform:
automation
Description of problem:
All automations are triggering twice. Below are two examples. One is triggered at Home assistant start, the other is triggered by platform time.
Problem-relevant configuration.yaml entries:
- alias: Värmepump - Automatisk värmejustering
  trigger:
    platform: time
    minutes: 0
    seconds: 00
  action ...

- alias: Autorun
  trigger:
    platform: homeassistant
    event: start
  action: ...

Traceback (if applicable):
Additional information:

I have still the same problem in 0.98.2 (installing 0.98.3 now). I'm using MariaDB. It is only the log that is shown double.


Escaping CSS class names for improved rendering

Fix: Rendering of CSS Class Names with Special Characters
In standard CSS naming conventions, class names can include alphanumeric characters, hyphens, and underscores. However, special characters require escaping to ensure proper rendering. This pull request introduces a regex-based solution that replaces special characters with their corresponding escape sequences. This fix guarantees correct rendering of class names containing special characters.
With this fix, the following code snippets can now be displayed correctly:

...

...

...

...

See https://github.com/yuanqing/create-figma-plugin/issues/209#issuecomment-2405244469


Service Discovery: make scheme selection more intuitive in un-specified case, add more tests

Make scheme selection more intuitive

If ServiceDiscoveryOptions.AllowAllSchemes is TRUE and the service name INCLUDES schemes (eg https://basket), we filter out results which do not have a matching scheme, applying the preference. i.e, https+http means pick only https endpoints if any https endpoints are present, http only if no https endpoints are present.
If ServiceDiscoveryOptions.AllowAllSchemes is TRUE and the service name DOES NOT INCLUDE schemes (eg, basket), then we should allow all endpoints.
If ServiceDiscoveryOptions.AllowAllSchemes is FALSE, the service name DOES NOT INCLUDE schemes, and ServiceDiscoveryOptions.AllowedSchemes contains some value (eg, ["https"]), then we should filter to include only endpoints matching those AllowedSchemes values.

In pseudo code, the logic is:
IncludeAll = AllowAllSchemes && ServiceName.Schemes.Count == 0
IncludedSchemes = ServiceName.Schemes.Count > 0 ? ServiceName.Schemes.Intersect(AllowedSchemes) : AllowedSchemes

Always check "default" config section first when no endpoint name is specified

When searching for a configuration section for a service and no endpoint name has been specified (i.e, the common case), we should first check the "default" section, then the sections corresponding to schemes. The current behavior is:

If the query specified any schemes, check those sections in order (https+http -> check "https" then "http" cfg sections)
If the query does not specify any schemes, check the "default" section.

The change is to always check the default section first if no endpoint name is specified.
Microsoft Reviewers: Open in CodeFlow

/backport to release/8.0


Using Microsoft.Azure.DocumentDB.TransientFaultHandling with 1.22.0

Does it make sense to use Microsoft.Azure.DocumentDB.TransientFaultHandling further with 1.22.0, as it seems the CosmosDB SDK For .NET seems to be tweaked for TransientFaultHandling internally.
So should we need the package any more with latest SDKs
https://www.nuget.org/packages/Microsoft.Azure.DocumentDB.TransientFaultHandling/

Equivalent of transient fault handling is in-build and on by-default in document db SDK. Also please note that TransientFaultHandling is very old and not actively developed and will be deprecated shortly.


noVNC improvements

Description
Continued from #2346
support scaling - add a control to enable scale of the VNC box (there should be 3 modes supported ~ none, local, remote)
Use case
(Man) Use case 1 - If I have a browser window 1600x1200, it shouldn't be fixed to the resolution set in the virtual machine e.g. 800x600, but be able to communicate the custom size and adjust on both sides --> as I result, the VNC content will try to fit as much as possible into 1600x1200, and the virtual machine desktop resolution will follow these dimentions. I.e., VM desktop will have a custom size of nearly 1600x1200. Browser VNC pixels will be always 1:1 to what is on VM desktop (no stretching, just desktop resize).
Use case 2 - Having a small VM desktop resolution leads to small VNC box. It would be great, if the VNC content can be just stretched to the size of window resulting just in a bigger image. Desktop size in the VM won't change, just the VNC image will be stretched.

Progress Status

[ ] Branch created
[ ] Code committed to development branch
[ ] Testing - QA
[ ] Documentation
[ ] Release notes - resolved issues, compatibility, known issues
[ ] Code committed to upstream release/hotfix branches
[ ] Documentation committed to upstream release/hotfix branches

covered with guacamole support in 6.0

covered with guacamole support in 6.0


Hlint redundancies.

See #1474 for redundant id, redundant return, redundant if, redundant <$>, redundant map and redundant lambda.

@mergifyIO rebase

@Kleidukos I rebased, got HLint running again (it was barfing on InterfaceFile, https://github.com/ndmitchell/hlint/issues/1442) and followed hints about rendundancies except for redundant brackets. They're numerous and I'll do those on a separate pull request.

Thanks for the review @Kleidukos. Closing here. I may create a new merge request for it over in the GHC repo.


Subtask 6: Implement the logic to delay the tracking for selected time.

Change the code to implement the delay for selected time in seconds. https://github.com/rilling/OpenTracksConcordia/issues/184

Duplicates #221


[stable/fluent-bit] Fix extraEntries indentation

Is this a new chart
No
What this PR does / why we need it:
Currently extraEntries values set to 8 that renders incorrect configuration file.
Indentation should be 4.
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/ok-to-test

Hi @zanhsieh, could you approve this PR please?

@gmlexx
Sorry I am not on the OWNERS file list. Below are owners:
@kfox1111
@edsiper
@hectorj2f
@Towmeykaw
Could you guys LGTM this PR please? Thank you.

ExtraEntries are used to add an entry to an existing section so it should be indented 8. If you want to add a whole new section you can use rawConfig.

@Towmeykaw indent 8 leads to incorrect fluent-bit config that is sensible to indentation. You can test it with the following config section.
rawConfig: |-
  @INCLUDE fluent-bit-service.conf
  @INCLUDE fluent-bit-input.conf
  @INCLUDE fluent-bit-filter.conf
  @INCLUDE fluent-bit-output.conf

extraEntries:
  input: |-
    [INPUT]
        Name        tail
        Path        /var/log/kube-apiserver-audit.log
        Tag         audit
        Parser      kubeapi_audit
        DB          /tail-db/tail-kubeapiserver-audit-state.db
        DB.Sync     Normal
        Refresh_Interval 5

Extraentries are for adding a single value to an existing section. Say you have the default tail
and you just wanted to add an extra value that is not provided then you can use Extraentries. If you want to add an entire new Input section you can add it under @INCLUDE fluent-bit-input.conf as a block of text.

Ok, got you, thanks.


Have to reenter credentials on Windows

I cloned a repo on Win10 64-bit and it worked fine.  However, when I tried removing a remote branch (git push origin :[branch name]) it asked me for my credentials again.
My version:
git version 1.9.5.msysgit.0
hub version 2.2.0

I noticed the same thing and in my case it seems to be due to a '|' in the password: hub stop aking for my password as soon as I type the '|' character

We're having some trouble on Windows, although we're not sure why: #815
From what I can tell, the password prompt is definitely busted. However, @twsouthwick said that he managed to input the password and it worked, but it wasn't persisted. The next time that happens, can you check that the successful password prompt did persist the auth credentials in $HOME/.config/hub? (I'm not sure what is the user's home directory on Windows.)
Also, please try with latest hub version v2.2.1, thanks!
/cc @jingweno

@Nicofuma I was able to type "|" on a 32 bit Windows with v2.2.1. Let me try on 64 bit as well. I'm using the Git shell comes with GitHub Windows. What's your terminal?
We could try upgrading the gopass lib and see if it fixes the issues. We're missing 2 latest Windows fixes.

@jingweno I will update hub and try tonight to see if I can reproduce the issue.

@jingweno I just reproduced it and actually it's not related to a specific char but to the password length: the prompt stop just after the 15th. So if your password has more than 15 chars you're stuck.
Reproduced with git bash and powershell (with powershell it even stops after the 13th)

I was able to reproduce the problem in Windows 8.1 Powershell with hub 2.2.1:
C:\Users\IEUser\Documents> .\hub clone playground
github.com username: mislav
github.com password for mislav (never stored):
←[36m> POST https://api.github.com/authorizations←[0m
←[36m> Authorization: Basic [REDACTED]←[0m
←[36m{"scopes":["repo"],"note":"hub for IE11Win8_1\\IEUser@IE11WIN8_1","note_url":"http://hub.github.com/"}
←[0m
←[36m< HTTP 401←[0m
←[36m{"message":"Bad credentials","documentation_url":"https://developer.github.com/v3"}←[0m
Error cloning repository: Unauthorized (HTTP 401)
Bad credentials

My password is longer than 15 characters. I tried to paste it in using Ctrl-V but it failed with output above. Then I thought that maybe Ctrl-V is not supported by Powershell, and I choose "Edit > Paste" from Powershell menu. And that worked!
I counted 2 hub bugs, though:

The verbose output contains ANSI escape colors that aren't supported in Powershell. We should not output color on Windows.

When you hub clone mislav/anyenv (a public repo) on a pristine system, you get prompted for authentication information, but even if that fails, the repo will be cloned anyway because all the information to guess the GitHub clone URL is on the command-line and API information isn't strictly necessary. However, the successful clone might fool you into believing that your authentication info was accepted and stored. But if authentication failed in the first place, no auth info will be stored and you will get prompted for authentication info on next usage of hub, which is confusing to the user.

I can reproduce it on a 64bit Windows 10 by typing a password more than 15 characters - 32bit Windows works okay though.
@mislav

The verbose output contains ANSI escape colors that aren't supported in Powershell. We should not output color on Windows.

The ANSI escape colors is fixable on Windows. I have been through similar fixes with go-colorable.

you get prompted for authentication information, but even if that fails, the repo will be cloned anyway because all the information to guess the GitHub clone URL is on the command-line and API information isn't strictly necessary

Yeah, this is pretty confusing. Will fix it as well.

I have the same problem when type the "0" of my password.
C:\Users\Rafael\workspace\vraptor4>hub pull-request -i 991 -b caelum/vraptor4 -h
 dobau/vraptor4 https://github.com/caelum/vraptor4/issues/991
github.com username: dobau
github.com password for dobau (never stored):
Error creating pull request: Unauthorized (HTTP 401)
Bad credentials

@dobau This should be fixed in https://github.com/github/hub/pull/927. And we'll cut a release for 2.2.2 soon. At the meant time, do you mind building the latest master and see if it fixes the issue?


To make them reactive, model contexts should be reworked to be observable

The model context at present is not observable. This means that if a variable in the model changes, derived variables do not automatically recalculate.
I would like to rework this so that every variable in the model context is observable and is calculated/recalculated as required.

Beyond this, each model context should carry a reference to it's parent. This way, it is possible to dynamically fall back to variables declared at a parent context.

I do not want to do this.  I will not support changing the variables (in a parent context) such that it will change which variables are in-scope for a child context.  Changing the values of those variables is fine (via observables at least), but changing which variables are available - no.


Keywords should be forbidden as namespace names

:rocket: Feature Request
Affected Languages

[ ] TypeScript or Javascript
[X] Python
[X] Java
[X] .NET (C#, F#, ...)
[X] Go

General Information

JSII Version: 1.37 and above
Platform:     all

[X] I may be able to implement this feature request

[ ] This feature might incur a breaking change

Description

When working with namespaces one can use names that are keywords in languages other than TS, resulting in invalid generated code. In our case it broke python when using the namespace lambda for AWS Lambda Resources.
Proposed Solution

Similar to other parts I'd expect an error to be thrown from JSII when a namespace with a keyword is detected.

It looks like there's a list of reserved words for each language: https://github.com/aws/jsii/blob/main/packages/jsii/lib/reserved-words.ts
It looks like this isn't used while generating namespaces.

Whoops, please re-open this one 😇


Move config items to configWorkspace

This pull addresses #18 and allows you to configure different Sass compilation settings for each project. Config items have been moved from config to configWorkspace; I chose to make every configuration workspace-specific instead of keeping some global, since some projects may have different needs.
I also added a boolean config option to enable/disable the extension per-workspace, so you no longer have to disable the whole extension.

Thanks @JohnWickham this is great. I have merged the request and will push the update to Nova shortly.


Spring security's concurrent session control feature is not working when max logged in sessions exceeds the limit

I would like to incorporate spring authorization server as an idp in place of keycloak. One requirement is it should support social logins along with basic authentication + need to control over max logged in user sessions across devices/browsers. I have defined max sessions as 1 and off the flag for max logins prevents when exceeds as per sessionManagement() spec. I read the documentation here and added these changes in demo example provided for social login, which is not working as expected.
In logs and local debug observed that authorization server able to expire the exceeded limit sessions (will expire old logged in sessions) but if I do any further application calls from that session, not getting any 401 error.
I have checked the documentation for this but didn't find anything on this. Can someone please assist on this? I'm using 1.1.4 version of spring authorization server and using spring cloud gateway as client.
Sample used for this https://github.com/sjohnr/spring-authorization-server/tree/bff-demo/samples

@maradanasai Questions are better suited to Stack Overflow. We prefer to use GitHub issues only for bugs and enhancements.
Also, Session Management is a Spring Security concern so make sure you tag it as a Spring Security question.


Update dependencies and workflows

Codecov Report
Patch has no changes to coverable lines.
:loudspeaker: Thoughts on this report? Let us know!.


Cube demo should resize on VK_SUBOPTIMAL_KHR

Based on this discussion, it looks like the Cube demo should resize on VK_SUBOPTIMAL_KHR in addition to VK_ERROR_OUT_OF_DATE_KHR.

Note that on Android, if you don't handle pre-rotation (most apps don't) then you'll get VK_SUBOPTIMAL_KHR every frame -- and so would definitely not want to re-create the swap-chain solely based on VK_SUBOPTIMAL_KHR being returned. :)

The "correct" solution, as you point out, is "handle the pre-rotation."  Or have an Android-specific "fix" with LOTS of comments as to why that occurs on Android and what the "real" fix is supposed to be.
Side note: Is there an example somewhere showing people how to handle the pre-rotation on Android?  That sounds non-trivial.
Vulkan demos and apps are supposed to be exemplars that people learn from.  If "official" code doesn't do it right, end users are stuck scratching their head figuring out why "official" code doesn't actually work when they try to apply it to their problems.

I did go back through the code, this bug seems to predate any Android port--so I can't blame them.  :)
It exists as far back as commit 5aefef01499d849a090b5af5143efce45bf0ebea which has no mention of Android in cube.c

It's easy enough to resize on VK_SUBOPTIMAL_KHR returned from QueuePresent, but it sounds like there is an Android consideration to take into account. Not being much of an Android programmer, can someone advise as to what should (or at a minimum, could) be done when vkcube running on Android gets VK_SUBOPTIMAL_KHR back from QueuePresent?


It would be good if you can determine the prefix from tslint.json and the look for angular-cli.json as user wants linted enabled in his project. So this should take precedence.
The following properties from tslint.json describes what should be the prefix for each of the declarable
"directive-selector-prefix": [true, "my"],
"component-selector-prefix": [true, "my"],
"pipe-naming": [true, "camelCase", "my"]

Closing for now, as it's design choice of this extension to be compatible to angular-cli


[DROOLS-1353] define ExecutableRunner interface

Replaced by https://github.com/droolsjbpm/drools/pull/1022


PlacePerpOrder2: Add flag to request relative expiry

This makes it easier to use TIF when the solana cluster clock does not match the actual clock.
It's not a perfect substitute, since relative expiry will be relative to transaction execution and not to transaction sending.

1/ lgtm, we could adapt the place_perp_order2 ix helper.. 2/ did the cargo audit always fail?

What did you mean about the ix helper?


[#776] Bump jsbeautify to 1.6.2.  Add support for collapse-preserve-inline

I have tested the package locally and the option seems to work fine now.

Failed tests?

Yeah, the unit tests appear to need some work.

Could I ask that you hold off on this for today?  There's been a request for moving preserve-inline to a separate setting to allow it to be combined with expand and end-expand.  If that happens, it would cause churn in this area.  Let me see how difficult that would be.

Please hold of on this for today.  Looking at options making this option less disruptive.

Hmm, never mind.  I don't have the bandwidth for further changes at this time.

Yup, probably because of the jsbeautify version bump.

+1 for this. It would be great to finally get this particular feature.

+1 if we can get this feature.

Thank you that's awesome!

Published to v0.28.23

After a short experiment, it seems that objects are sitll expanded to multiple lines as before although this new option is selected. Can someone else share experiences?

@ttsirkia please create a new issue and follow the instructions provided in the template. Thank you.

Sure, #831. I was just asking if someone could give a quick comment if there is only something wrong in my environment.


Issue with Cycling Cameras

When cycling the cameras (CrossVonage.Current.CycleCamera()) and exiting the page, then returning to the page, the camera no longer functions. If you cycle the camera only once, it partially functions, but with a greatly reduced framerate and with a general lagginess to the app, and if you cycle it any more eventually the app will crash altogether. with the error code:
CAMERA_DISCONNECTED (2): checkPidStatus:2443: The camera device has been disconnected'
I've been using the old version of your project for a few months now and was taking a look at this again since I ran into this issue; it seems to happen on both the old CrossOpenTok and new CrossVonage versions. From my own research it seems like this might be something having to do with DreamTeam's OpenTok.Android library, but I'm not sure since I couldn't find any other related issues.

Hi @ross8alex and thanks for your report.
Unfortunately, I should say that I can't do much cuz it actually looks like the issue is not here (this plugin just calls CycleCamera of PublisherKit which is inside DreamTeam's lib... or even in native code)
I propose to move this issue to DreamTeam's repository (especially if the issue is reproducible w/out my lib

Ok I'll see if I can replicate the issue using DreamTeam's lib, thanks for your help.


chore: release 4.2.5

:robot: I have created a release *beep* *boop*
4.2.5 (2021-05-28)
Bug Fixes

loop copy (eb34bd7)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/mili-project-manager/mili/releases/tag/v4.2.5 :sunflower:


Persisting memory state between requests in FastAPI

First Check

[X] I added a very descriptive title to this issue.
[X] I used the GitHub search to find a similar issue and didn't find it.
[X] I searched the FastAPI documentation, with the integrated search.
[X] I already searched in Google "How to X in FastAPI" and didn't find any information.
[X] I already read and followed all the tutorial in the docs and didn't find an answer.
[X] I already checked if it is not related to FastAPI but to Pydantic.
[X] I already checked if it is not related to FastAPI but to Swagger UI.
[X] I already checked if it is not related to FastAPI but to ReDoc.

Commit to Help

[X] I commit to help with one of those options 👆

Example Code
from fastapi import FastAPI

class GlobalTest():
    __count = 1

    @classmethod
    def main(self):
        self.__count += 1
        return self.__count

app = FastAPI()
@app.get("/")
async def read_root():
    return GlobalTest.main()

Description
Open two browsers. Go to <IP_ADDRESS>/docs and call this API a couple of times. You just got a counter over different requests. The static class is memorizing the value over different requests.
Is there any solution to this? Is there any other way to create static classes that will not persist over different requests? I think its definitely an issue, and potentially a security concern.
Thanks
Operating System
Linux
Operating System Details
No response
FastAPI Version
0.68.1
Python Version
3.9.7
Additional Context
No response

I don’t understand the issue, this is very much by design. If you don’t want to share state of a class, then don’t use static methods? This is what they are supposed to do, and is certainly not a security concern (unless you don’t know what you are doing, but then everything is a security concern).
Maybe it helps if you explain what you are trying to achieve using a classmethod, and we can give you some pointers?

@pvahabi pvahabi
This is probably not the thing you want to do.
If you need some kind of state to be saved you should use database or redis or something else.
Your example should work, but only with 1 worker, as soon as you get 2 workers each will get his own memory space and you will have 2 GlobalTest classes and 2 __count properties so you will have 2 states stored.
Storing things is memory usually is bad because your service is not stateless anymore, you can not scale it and what happens with data if service restarts - you lose your data.

Hey @JarroVGIT and @zoliknemet . Thanks for answering, let me give more insights.
Is there a way to be in a totally stateless service? This situation would never ever be possible in a server with PHP / Apache. There is no way that one request is changing the state of a class and it will be kept in memory. It is totally stateless.
The issue is caused by uvicorn workers being used for more than one request before they are recreated. Upon subsequent requests a single worker may have some memory state such as static or global variables carried over from previous requests (which is bad! its useless, and it make the service not anymore stateless). This could be avoided by setting uvicorn to one worker per request, but it would be too slow.
Any suggestion?
@zoliknemet your solution is not bad, but we have many programmers, and I would like to achieve a situation where there is no way that you can not be in a stateless situation.
@JarroVGIT did you expect all request would create their own FastAPI()? Not really. I'm just expecting that the static classes internal variables if changed will remain isolated to the current request (as in any fully stateless service, such as PHP/Apache)
ps: We have a more complicated code, this is just a dummy example.

@tiangolo would you be able to look at it? the example code on the top should be self explanatory of the issue.

@pvahabi
do you know who is using your app, do you have some kind of authentication?
If you don't can you generate some kind of ID per session, some UUID on first connect to your app, than you will know each tab that is unique. Look at example:
from uuid import uuid4
import uvicorn as uvicorn
from fastapi import FastAPI, Header

app = FastAPI()

class GlobalTest:
    __count = {}

    @classmethod
    def main(self, identifier: str):
        if identifier not in self.__count:
            self.__count[identifier] = 1
        self.__count[identifier] += 1
        return self.__count[identifier]

@app.post("/anonymous-register")
async def read_root():
    return uuid4().hex

@app.get("/count")
async def read_root(unique_session_id: str = Header(...)):
    return GlobalTest.main(unique_session_id)

if __name__ == "__main__":
    uvicorn.run(app)

Now each time CLIENT connects, goes to /anonymous-register and gets UUID which represents him. For each other request just send that UUID in header
Try to give me more info about use case (why you need it)
Do you have any Frontend or is it just API

Not really. I'm just expecting that the "static classes internal variables" if changed will remain isolated to the current request (as in any fully stateless service, such as PHP/Apache)

But you actually did expect it, as you didn't want to share state across requests. I don't know how to help to be honest, as this is normal behaviour for python based web services (both WSGI based such as Flask as ASGI based, such as Starlette and (by extension) FastAPI).
PHP, once designed as an templating mechanic, works differently and is reinterpreted per request. Starlette, not so much so, it is "running" while talking 'events' with uvicorn. Each worker gets its own process (and by that, its own instance of the application) so local state of the application is not shared across workers but is shared by the requests. It's basically how you build ASGI based applications, and I don't see any way around it (other then; just not use it). It is not a FastAPI issue though, maybe you have more luck in the Starlette repo? They know this stuff better than I do for sure :)

Is there any solution to this? Is there any other way to create static classes that will not persist over different requests? I think its definitely an issue, and potentially a security concern.

It's not a security concern.

This could be avoided by setting uvicorn to one worker per request, but it would be too slow.

That's not true. It cannot be avoided. Those are python processes. The application is not stateless because you're coding it as non-stateless, it's not a web framework or server issue, it's just an implementation issue.
I understand what you mean by this issue, but there's a misconception that the web framework (or server implementation) should prevent you from doing what you are able to do, but that's not the case. The worker is a python process, and objects are living there, if you change the value of an object, then that's it, the value will change.

@Kludex hey Marcelo thanks for the answer, however this is still an issue.
As an example look at this: https://stackoverflow.com/questions/67663970/optimal-way-to-initialize-heavy-services-only-once-in-fastapi
Think about heavy objects you want to initialize once. Now add an internal variable to the heavy objects.
Can you guarantee that multiple requests in parallel using the methods of this objects will not have any interference between each others?
Req 1
while still executing (not finished yet) is changing multiple time an internal variable of the heavy object.
Req 2
while still executing (not finished yet)  is reading an internal variable of the heavy object.
In this context, it seems to me the only way to be safe, its to create a new instance of any, literally any object per each request. Can you elaborate?
Thanks

@JarroVGIT, @Kludex, I came here from Reddit thread (140+ comments) where some suspicious people stated that FastAPI has a lot of unanswered and unfixed bugs, memory leaks, etc. (without providing any evidences) and that some previously unknown for me "Starlite" framework does fixed all those issues and nearly perfect. So I've came here in an attempt to find such PRs and/or opened issues by myself. And I found at least mostly, only trash PRs and strange "issues" like this - when a reporter has no any knowledge even in basic concepts of tools which they trying to use. I mean even basic Python knowledge. And I can understand when a such "issues" came from some non-tech person (former designers, etc.): they may simply not be trained in the skill of working with tools and information (such a skill can be learned, for example, while acquiring some technical degree). But! Professor of AI from Berkeley with a PhD in CS??? Are you series??? It... is... completely outrageous...
Dear maintainers, my sincere condolences that you have to delve into the hundreds of components of this terrible ungrateful and outrageously arrogant swamp. Thank you for the framework and all the work along the way - you are awesome!

Thank you for the framework and all the work along the way - you are awesome!

I thank you for this, but there's no need to fight written violence with more of it.
I'm grateful for all the people that asked questions here. Most of the questions could have been on StackOverflow, but we actually tell people to create issues for questions (it's on our documentation). The reason for it is to reward the people that help.
There were many side effects in my life because of those questions: I learned a lot because I wanted to understand each question, and find the answer, even if they look easy now, they weren't when I started helping; and because of that, I grew my knowledge, and I was able to help other projects that help FastAPI, like Starlette and Uvicorn.
Let's try to see things from a different light. :pray:

@Kludex , Understood. The approach deserves respect! I've deleted that my comment. I'm more concerned that this approach, and as a result, hundreds of unclosed pull requests and issues, are used by some strange people as фт evidence that FastAPI is problematic, like its authors. And also, perhaps, to advertise their own framework: look how everything is fine with us, but everything is terrible with them. Just FYI: https://www.reddit.com/r/Python/comments/z4nqb4/falcon_vs_flask/ . I am confused by Starlite motives: they are some cyber security company and such companies are almost automatically suspicious for me (one "cyber security company" tried to hire me some time ago and I saw that they are botnet-managers when I came to their "kind of an office" in order to remove inadequate contract clause).
(Feel free to remove this comment when it will be not needed anymore)


Add option to read a mapping from a Delphi installation's .dcu files

Currently Unit2NS has the facility to read mapping details from a Delphi installation by reading the names of .pas files in the source directory. Is it possible to provide the option to read the precompiled .dcu files from the object files directory?

Method that gets .pas files is:
https://github.com/delphidabbler/unit2ns/blob/37465c01d770eb1155cfb77cc9281bec627d369c/src/UDataIO.pas#L425
Do we need to pass required extension as a parameter?

Implemented by merge commit ed46f06


Confusing error message when exporting invalid main to platform and defining a type alias

Weird edge case here:
app "day 1 - bug report"
    packages { pf: "https://github.com/roc-lang/basic-cli/releases/download/0.1.1/zAoiC9xtQPHywYk350_b7ust04BmWLW00sjb9ZPtSQk.tar.br" }
    imports [ pf.Task, pf.Process ]
    provides [myMain] to pf

MyThing : {}

myMain : Task.Task {} []
myMain =
  Process.exit 0

Throws this error:
── UNRECOGNIZED NAME ─ ...zAoiC9xtQPHywYk350_b7ust04BmWLW00sjb9ZPtSQk/main.roc ─

The #UserApp module does not expose anything by the name MyThing.

── UNRECOGNIZED NAME ─ ...zAoiC9xtQPHywYk350_b7ust04BmWLW00sjb9ZPtSQk/main.roc ─

The #UserApp module does not expose anything by the name MyThing.

However, renaming myMain to main makes it compile without any warnings or errors.

Problem still exists. Up to date reproduction:
app [myMain] { pf: platform "https://github.com/roc-lang/basic-cli/releases/download/0.16.0/O00IPk-Krg_diNS2dVWlI0ZQP794Vctxzv0ha96mK0E.tar.br" }

MyThing : {}

myMain : Task {} []
myMain =
    Task.ok {}

error:
── UNRECOGNIZED NAME in ...00IPk-Krg_diNS2dVWlI0ZQP794Vctxzv0ha96mK0E/main.roc ─

The #UserApp module does not expose anything by the name MyThing.

── UNRECOGNIZED NAME in ...00IPk-Krg_diNS2dVWlI0ZQP794Vctxzv0ha96mK0E/main.roc ─

The #UserApp module does not expose anything by the name MyThing.

────────────────────────────────────────────────────────────────────────────────

2 errors and 1 warning found in 76 ms

Renaming to main compiles and runs, though now there's a warning that MyThing is unused (which is correct and expected).


[choreolib, docs] ChoreoLib docs overhaul

The primary motivation here is to migrate to more of a tutorial style of documentation, as well as fix the inconsistencies with recent API changes.
Primary changes:

Article reorganization
The trajectory controller is now documented in the "Getting Started" section, promoting discoverability, the Trajectory API now has its own article
Differential drive control is now documented
Trajectory API docs have been expanded to show proper use of a trajectory with a trajectory controller
Major overhaul to the AutoFactory documentation

Majority of the documentation is now game agnostic, outside of branching autos
Improved documentation of individual features of the AutoFactory (instead of large code blocks)
Removal of large preamble in favor of generic subsystem names ("intakeSubsystem", "scoringSubsystem")
2024 example now includes a graphic for pickup / scoring locations

Looking for feedback, some of the writing still feels too verbose.
To do:

[ ] Missing C++/Python code examples
[ ] Odometry resetting example code (pending possible API change)
[ ] AutoChooser example code (pending #949)

Here's one of the modified trigger warning blurbs that u should probably add
!!! warning
Triggers can have "hygiene" issues if not used correctly.
If you want to use triggers not created by an AutoRoutine or AutoTrajectory,
you must do one of the 2 things:

Place them within an and() or or() call from a trigger
created by an AutoRoutine or AutoTrajectory.
For instance, path1.active().and(externalTrigger).onTrue(...)
Use routine.observe(trigger) or routine.observe(BooleanSupplier
if a trigger is not enclosed within an and() or or statement.
If this is not done, triggers intended to be auto-only can activate during teleop.
For example, this can cause a check intended for intaking a game piece during an auto
to also be checked and reacted to during teleop.

Ah, meant to put adding that note on the to do


Doesn't work on preview 6?

A lot of errors when I try to install it on new project:
https://i.imgur.com/Bg964WC.png

not updated yet


Should make it clear which tutorials require a v10 fixpack

At the moment it is not clear which tutorials require the end user to install a later fixpack of Integration Bus.
Suggest that each tutorial should have in its brief description (which you get when clicking on its name) whether it requires a particular fixpack level.  There are a few like this:

callable flows needs FP4
BTM needs FP3
calling Global Cache from a map needs FP2
Should also update the details.html for each of those individual tutorials to say this, which the Global Cache one already does but the others do not.

I have just submitted a pull request to fix this, issue.


Add maxDepth param into find operations

maxDepth: 0 -> relations are disabled
maxDepth: 1 -> only relations attached to the main entity
maxDepth: n -> ...
Add also a default value as the limit (pageSize)

Not very useful since in graphql you can specify a max depth.

The implementation is not trivial due to the data loader.

What do you mean with "not trivial"?
Il giorno lun 21 feb 2022 alle ore 15:49 Edoardo Barbieri <
@.***> ha scritto:

The implementation is not trivial due to the data loader.
—
Reply to this email directly, view it on GitHub
https://github.com/twinlogix/typetta/issues/77#issuecomment-1046956775,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAD7YBG67PCYOASNMFWYJUTU4JGHFANCNFSM5MMZVYUQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

--
Mattia Minotti
*Head of Engineering *at TwinLogix S.r.l.
TwinLogix S.r.l.
P.IVA<PHONE_NUMBER>9
www.twinlogix.com
via Dante di Nanni, 29
47822 Santarcangelo di R. (RN)
via Piave, 48
Cesena (FC)


Addresses receiving 195.94902 SAFE airdrop are from the same guy

Related Safe Addresses
0x03e8cd1783b7e04518ec14a15b37741ee6a08f1d
0x081edd86c296091f723a698084d18ddb3f3edd9c
0x1310f4ef504d21bb774937afc8275863afe1bce5
0x1510b54643541e731767000f9ae93a7369988eb7
0x1b228a7d1550b47b6d609e4937a39038e1c8e6d2
0x1fae4cf5345ea71e0e5785d8dbc093571a9a332a
0x3f7f04ab5ad9970b2d710cca375fcd28eafeeab4
0x45701312ad912bcd336729e0ca6a1db061af0222
0x4f9d6f8b453a198ec93142238ebeb328ad99acc2
0x52d1ec1fcca9271afeaa18c4df25b22cac6f1b61
0x5e95c302b8b8bcc4cd0f9fe09bef84718bed9e73
0x6312e6d2a7324fd6ccf8d49a2d3be5260fc330ad
0x68e591f84678dc2534286c4a0ecdb044dc4eb91d
0x693fd64d21723c1a95dc5ecee70a256ce97c9a9c
0x69e77cefab916cb47e35f412f8dab1ed2a415c05
0x79a9969e98be9c9bde852cf7d81634f4eb817eeb
0x79d5ece4d76f1b779426442966f13ed77309c489
0x7a4ad59eb4a37033762f4dabf091d6c4c109b03c
0xa2bab96a41bd86ebd49936c9cf9f06981ce03ffb
0xa3f8405ce6d06bd2134aabd617371f37007c23f2
0xa55c367f1258c97778d3cb632849eb8b070390a1
0xa5a141bdbbcdb0a2e8d26a015c8e57a170f6ac2c
0xaca913eb567fb86c319a42bb504f464701e45633
0xad00293b3bbe2281bab9d8112ddc3df3b517d2e6
0xb106fa9a8080ea975e36b57968e94d3269d4f5a1
0xc2e0283da19df3d8e439fcae68b05f8893b7fe0d
0xc7de22f729bd92faa6a6a420808db0e6fdfd7863
0xefcebb7c99d29f79a79929595e4d13c2796e2dbf
0xefdd3e15971722897e726f324144e78e2083ee2a
0xf125c3543272b5d4b3e94f7f456eec035ed44b13
0xf1fb747189632a1d6ab99b422cc07e09e52d5291
0xf53a9356ba89e8a127d8bf5e85e7b0d0bd3cf104
0xff06efdf05882c2268cb051f32d568b13cf7aef5

Reasoning
These SAFEs were created on Jun-05-2022 UTC. They only have 1 tx history happened on that day, and that tx is identical for all of them:
for example:
https://etherscan.io/tx/0x2b9312279ee74c333280cdbb6b5d61f35726d2bf4706caa0ecf984506f000127#eventlog
https://etherscan.io/tx/0xfcd18adb5d655f81c4c76b3f0802af271bd3940b8a7eee04efe5ccf0dfcf2929#eventlog

it approved a token on Uniswap.
If we check the owner addresses who created these safes, their activities are identical, too. For example:
https://etherscan.io/address/0xb334de1bee7bef8cce8f0d37aae0e7b41dff5620
https://etherscan.io/address/0xe4e244f8de69e719049f252c8b890e17ff59c962
They all claimed 5798 RADAR token airdrop on Dec-18-2021, they all made deposits into arbitrum on Sep-11-2021, and they call claimed One Day Punk NFT on Sep-04-2021
Methodology
I sorted the excel sheet by the amount of eligible SAFE tokens. I identified that there are some addresses eligible of exactly the same amount of tokens. After manually checking etherscan tx history, I found these addresses with exact same activity pattern.
Since the airdrop amount is determined partially by the balance of SAFE, it very unlikely that different SAFEs have exactly the same amount of airdrop. This is a useful clue.
Safe Address
eth:0x984129b1C8D6048DF516D81f730475AF7D0E4223

Why did I say the tx approved a token on Uniswap?
It's found from the tx log, "Approval" event is for 0x7a250d5630b4cf539739df2c5dacb4c659f2488d which is Uniswap V2 router

Thanks for the report @wjw12, we've verified that the following addresses seems to be airdrop farming.
0x03e8cd1783b7e04518ec14a15b37741ee6a08f1d
0x081edd86c296091f723a698084d18ddb3f3edd9c
0x1310f4ef504d21bb774937afc8275863afe1bce5
0x1510b54643541e731767000f9ae93a7369988eb7
0x1b228a7d1550b47b6d609e4937a39038e1c8e6d2
0x1fae4cf5345ea71e0e5785d8dbc093571a9a332a
0x3f7f04ab5ad9970b2d710cca375fcd28eafeeab4
0x45701312ad912bcd336729e0ca6a1db061af0222
0x4f9d6f8b453a198ec93142238ebeb328ad99acc2
0x52d1ec1fcca9271afeaa18c4df25b22cac6f1b61
0x5e95c302b8b8bcc4cd0f9fe09bef84718bed9e73
0x6312e6d2a7324fd6ccf8d49a2d3be5260fc330ad
0x68e591f84678dc2534286c4a0ecdb044dc4eb91d
0x693fd64d21723c1a95dc5ecee70a256ce97c9a9c
0x69e77cefab916cb47e35f412f8dab1ed2a415c05
0x79a9969e98be9c9bde852cf7d81634f4eb817eeb
0x79d5ece4d76f1b779426442966f13ed77309c489
0x7a4ad59eb4a37033762f4dabf091d6c4c109b03c
0xa2bab96a41bd86ebd49936c9cf9f06981ce03ffb
0xa3f8405ce6d06bd2134aabd617371f37007c23f2
0xa55c367f1258c97778d3cb632849eb8b070390a1
0xa5a141bdbbcdb0a2e8d26a015c8e57a170f6ac2c
0xaca913eb567fb86c319a42bb504f464701e45633
0xad00293b3bbe2281bab9d8112ddc3df3b517d2e6
0xb106fa9a8080ea975e36b57968e94d3269d4f5a1
0xc2e0283da19df3d8e439fcae68b05f8893b7fe0d
0xc7de22f729bd92faa6a6a420808db0e6fdfd7863
0xefcebb7c99d29f79a79929595e4d13c2796e2dbf
0xefdd3e15971722897e726f324144e78e2083ee2a
0xf125c3543272b5d4b3e94f7f456eec035ed44b13
0xf1fb747189632a1d6ab99b422cc07e09e52d5291
0xf53a9356ba89e8a127d8bf5e85e7b0d0bd3cf104
0xff06efdf05882c2268cb051f32d568b13cf7aef5


Importing reach causes exception in IE 11 w/ polyfill

Possibly related to #214.
I built an app with create-react-app.
Since my app needs to support Internet Explorer 11, I'm pulling in the default polyfills from polyfill.io.
If I attempt to use @reach/router with my app, I get this error:

SCRIPT445: Object doesn't support this action
polyfill.js (2784,3)

This error occurs on the first line of the body of the getOwnPropertyDescriptor(o, key) polyfill. It appears this polyfill is getting redefined by polyfill.io even though it seems to already be native to IE 11's JS engine.
Steps to reproduce:

Create a new project with Create-React-App polyfill-fail

Change public/index.html to this:

<!DOCTYPE html>

        <meta charset="utf-8" />
        <meta name="robots" content="noindex" />
        <meta name="viewport" content="width=device-width, initial-scale=1, shrink-to-fit=no" />
        <meta name="theme-color" content="#000000" />
        <link rel="shortcut icon" href="%PUBLIC_URL%/favicon.ico" />
        <link rel="manifest" href="%PUBLIC_URL%/manifest.json" />
        <title>Polyfill Fail</title>

        You need to enable JavaScript to run this app.

Replace src/index.jsx with this:

import { navigate } from "@reach/router";
import React from "react";
import ReactDOM from "react-dom";

const rootElement = document.getElementById("root");
ReactDOM.render(<App />, rootElement);

function App() {
    return (
        <>

Hello React!

            <button
                onClick={() => {
                    navigateRelativeToRoot("/");
                }}
            >
                Activate Lasers
            </button>
        </>
    );
}

function navigateRelativeToRoot(relativeUrl) {
    const baseDirectoryName = "";
    const url = baseDirectoryName + relativeUrl;
    navigate(url);
}

(Apologies for not providing a codesandbox example--it doesn't appear to work in IE 11.)
Discussion
If you remove the polyfill script, the page will load properly, but then if you click the button, you will get an error that Promise is not defined.
If you reinstate the polyfill and remove navigateRelativeToRoot, the corresponding import, and the button, the app will run fine.
So, there is a problematic interaction between @reach/router and the polyfills needed to run the app in IE 11.
Edit
I've now added a github repo that reproduces this bug.
Steps:

Clone to your machine
Open a command line to the root directory
npm install
npm start
Open IE 11
Hit F12
Browse to localhost:3000
Note the errors in the console

Closing because most likely not related to @reach/router.
I suspect https://github.com/ljharb/es-abstract/issues/39 is the culprit.


Display search box only if the list has 10 items or more ?

It is possible to create a option to show the search only in cases where the select has 10 or more items (for example), because doesn't make sense show a search box for 3 items .

I saw replys about set a property to negative number, but it hide the search box without check the number of items on list

https://select2.org/searching#limiting-display-of-the-search-box-to-large-result-sets
"Select2 minimum results for search"
That same option that you can set to Infinity to hide the search box permanently, you can also set to a reasonable number to only hide the search box for small numbers of options.


Minor fix for properties names inconsistancy

@Hasys
The code change looks OK, but I'm not aware that it changes anything in the UI.
In the properties pane for activities, links etc., there is a property 'Font color', in which the use of lowercase 'c' in 'color' is inconsistent with other property names. It's probably worth creating a BZ for this inconsistency in property names so that all properties can be made to use consistent case.
(To change the property name in the UI,  it needs to be changed at
https://github.com/droolsjbpm/jbpm-designer/blob/master/jbpm-designer-api/src/main/resources/org/jbpm/designer/resources/i18n/DesignerConstants.properties#L1273 )

@jlindop thank you for your notion. This PR fixed, new bugzilla created: https://bugzilla.redhat.com/show_bug.cgi?id=1287000

@Hasys This looks good, and thanks for creating the new BZ.
+1

master: 88db906e579e1310c04825312680b85327a5b80b


Remove boolean options fallback

While unnecessary since all of them have a default value
Having build-bundle set to false was is fact turning it to true through the fallback

oh, good catch

i wonder why the ncc build diff is so huge this time

i wonder why the ncc build diff is so huge this time

I don't know, I just install ncc and done the command. Should I done another npm/yarn command before ?

I think I found the issue, I think need to use yarn install --pure-lockfile

Dist files should be fixed and good now.


icon font from iconfont.com not work

I have try serval to make it work, but until now i cannot solve it .  The methods as blow :
1
I download the iconfont.*(eot,svg,ttf,woff) resources and copy them in the dir src/assets/chat(chat is the new folder name) .
I put the iconfont.scc in the src/theme.
I want to use the resource in the tabs.html , so according tot the structure of project generated angular-cli , i update the tabs folder under the src/pages/ and create the new file named tabs.scss.
Then in the tabs.html . i refer the style as blow :
<ion-tabs>
  <ion-tab [root]="tab1Root" tabTitle="Home" tabIcon="chat"></ion-tab>
</ion-tabs>

But the style not show. the screenshot as below :

2
i try to put the iconfont.* resource into the folder www/assets/. and no luck .

Hello! Thank you for opening an issue with us!
The Ionic Framework issues list is designated for the reporting of bugs with the Ionic Framework or for requesting new features. As this seems like more of a support question I am going to close this issue.
I urge that you ask this question on our forum or on our slack channel.
Thanks for using Ionic!


Allow for multiple instances of Amplify DataStore

Is your feature request related to a problem? Please describe.
Feature request to utilize the DataStore implementation (even just locally) in multiple instances in an application. Currently internal decisions (like datastore name based on bundleId) make this difficult.
Describe the solution you'd like
Ability to give datastore an identifier (perhaps defaulted to the appId configured for the amplify instance?)
Describe alternatives you've considered
Even bundling multiple instances in different frameworks doesn't get around filesystem issues like the datastore naming.

Hi @johnm555. The Amplify architecture defines that installed plugins are used through the category facade (i.e. Amplify.DataStore, Amplify.Storage, etc). Therefore, having multiple instances of a plugin is not really supported and not a feature we're currently evaluating.
That being said, I would like to understand your use case. Maybe there's another way, or even another feature in our roadmap that could help you achieve the same goal. If you're able to provide us more info about what you're trying to achieve, I would be happy to assist you.

Our application supports many different independent feature sets and in order to help us scale some development & operations, we feel that at times it would be easier to have different DataStores (even if that means accessing it outside of the Amplify.DataStore shorthands and configuring/accessing plugins explicitly. Things like the datastore path being based on the bundleId make this difficult.

@johnm555 thanks for the follow up. Even if we allow the datastore path to be customized, using DataStore straight from the plugin class isn't something we recommend or even plan to support. The Amplify.DataStore is the only entry point we can guarantee that won't break in future releases - not without a proper deprecation plan in place.

That makes sense. What about supporting an application that would have two different frameworks with their own integrations with Amplify datastore?

Another extremely related issue is that it seems the dataStore configuration does not allow which apiName to specify if multiple APIs are configured.

That makes sense. What about supporting an application that would have two different frameworks with their own integrations with Amplify datastore?

Not sure I understand, what do you mean by "two different frameworks with their own integrations"?

Another extremely related issue is that it seems the dataStore configuration does not allow which apiName to specify if multiple APIs are configured.

This is good feedback, thanks! Do you mind creating another feature request with this? You can create a new issue from your comment by clicking the 3 dots and "reference in a new issue".

Multiple instances of DataStore is quite a heavy lift that requires a lot of testing to ensure it is still performant, especially performing multiple sync processes on DataStore.start(). See https://github.com/aws-amplify/amplify-swift/issues/2723#issuecomment-1412455546 comment. I don't think we have any future plans in taking on this feature request. Please let us know if you are still blocked on your development since it's been a few years now, and we can explore workarounds.


Consider including logger metadata as tags in reported errors

Elixir's Logger supports the concept of metadata, which are arbitrary key-value pairs that, if configured, will be included in all logged messages in a standardized format.  In our application we have some standard things that define the context for every process (such as user) which we include in Logger.metadata -- that way, every logged message includes them in an easy-to-parse, standard format.
Unfortunately, since we don't also include those bits of data in our exception messages, we don't have that context when looking at errors reported to Sentry.  That makes it hard to troubleshoot errors since the context in Logger.metadata is so important to understanding the error (we need to know, for example, which user's data triggered the error).
It would be a huge help if Logger.metadata was included in errors submitted to sentry.  It seems like it would fit naturally in tags.  Is that something you would consider?

Thanks for opening this 🙂
I'm not super familiar with Logger metadata, but it is stored per process?  If that is the case, you should be able to do something like:
Sentry.Context.set_user_context(%{user_id: 34, username: "mitchellhenke"})

in the same place you set Logger metadata, and have that information appear in Sentry:

Sentry context is also set per-process, so they should work somewhat similarly.  If the above does solve this for you sufficiently, the feature is less compelling.  If not, I can take a deeper look.

I'm not super familiar with Logger metadata, but it is stored per process?

Yep:
https://github.com/elixir-lang/elixir/blob/v1.4.2/lib/logger/lib/logger.ex#L307-L309

If that is the case, you should be able to do something like:

I wasn't aware of Sentry.Context, so thanks for that tip!  This would definitely work, but would be pretty sub-optimal for us, for a few reasons:

We set Logger.metadata in lots of places.  (In my project, ack Logger.metadata) returns 29 instances).  It would be a bunch of work to update them all (either to call both Logger.metadata and Sentry.Context.set_user_context or to call a new function that delegates to both).
In our umbrella app, we have a few packaged deployable apps, and then lots of mini-apps that act as internal services.  Sentry is only directly depended upon from the packaged deployable apps (as it's basically just a strategy for how errors are reported).  I don't really want to add a dependency from all of our sub-apps (currently 44 and counting!) on sentry just to make sure our process context is stored in both the logger and in sentry.
We have some utilities that we use when spawning new processes (such as tasks) that copy logger metadata form the parent to the child process so that the metadata is available when logging messages from the spawned process.  We'd have to also have it copy Sentry.Context data as well, but it seems sub-optimal to have to always copy both.

Since Logger is part of the Elixir standard library, and, in my experience, is the standard way to store process context metadata, it would be ideal if this library could use it automatically (or when a user opts-in via a config setting); that way every sentry-elixir user who uses Logger.metadata like we do does not have to make tons of updates to their application just to make their metadata available to Sentry.
I think that implementing this would be as simple as making Sentry.Context.get_context merge what it's stored in the process dictionary with what Logger.metadata/0 returns.  It probably makes sense for it to be an opt-in feature, so that users aren't surprised by the logger metadata always being included, and also to guard against cases where Logger isn't running.  While it's part of the standard library, it has to be included as an application in releases (e.g. with Exrm or distillery) to be usable at runtime in production, I believe.  (That's why mix new includes it in the generated mix.exs).
Thoughts?

Thanks for explaining, it does seem difficult to fit current functionality into that use-case.  If you don't mind, I have a few more questions 🙂
There are different 'contexts' within Sentry that the metadata key/values could be added to (User, Tags, HTTP, and Extra).  SDK Docs for that are here.  Simple case would be merging the Logger metadata into the Extra context.  Would that be sufficient?  You wouldn't need to pick which context it should use or specify which keys from the metadata to take?

That's a great question.  I'm not familiar with how the sub-contexts are used (e.g. how is the data within them used and surfaced within the Sentry UI?).  Tags I understand (we use them a bit) so it's easy for me to understand how they are use and I was originally thinking Logger.metadata would just be merged into that.  It's not clear to me how extra context is used--can you explain?
What might be best is providing a useful, simple default (e.g. merging all metadata into tags or extra, depending on what makes more sense), and then support users optionally configuring a custom merge function, which could be used by the user to merge different parts of Logger.metadata into different contexts based on their application's needs.

Is that something you think could fit for your application, or is that too much flexibility?

I quite like it!  One potential issue: since the Logger.metadata is stored in the process dictionary, that function will have to be called from the process from which the event originates.  If it gets called in a spawned task (or whatever), Logger.metadata will be empty.
I'm also not sure if the lambda form will ever work for people who deploy their app as releases with a tool like ExRM or distillery.  Such tools compile all mix config down to a sys.config file of erlang terms and I'm not sure if it supports anonymous functions in there.  Sounds like you plan to support MFA, which is a nice work around, but you may want to emphasize as the primary case so that users don't try the lambda, have it work in dev, try deploying, and have it fail.
Finally, the name pre_event_send_function could be better, IMO.  Maybe something like before_send_event ?

We allow for setting tags context on a global basis through config, but otherwise both Sentry.Context and Logger.metadata are limited to in-process storage without much we can do on the Sentry side as far as I understand.
I 100% agree MFA should be the primary option, and much prefer your naming.
Thank you for taking the time to discuss and help me work this out!  I plan on putting a PR together for it today or early next week.  You're of course not obligated to, but I can mention you on it if you'd like to take a look before it gets merged. 🙂

We allow for setting tags context on a global basis through config, but otherwise both Sentry.Context and Logger.metadata are limited to in-process storage without much we can do on the Sentry side as far as I understand.

Right, there's definitely not much you can do in general, but there is something you could do that would make this solution completely unusable for our use case: wait to use this configured function until you are in the spawned task that is used to send events to sentry in the background.  If you did that, this config function would never work for our use case.  As long as you avoid that, it should work fine as long as our sentry events originate from processes with Logger.metadata set.
That's the only point I was trying to make.

You're of course not obligated to, but I can mention you on it if you'd like to take a look before it gets merged.

Sure!

Right, there's definitely not much you can do in general, but there is something you could do that would make this solution completely unusable for our use case: wait to use this configured function until you are in the spawned task that is used to send events to sentry in the background. If you did that, this config function would never work for our use case. As long as you avoid that, it should work fine as long as our sentry events originate from processes with Logger.metadata set.

Yep, sorry, I didn't communicate it well.  That's the plan.


Turn off shallow cloning.

This PR turns off shallow cloning in. I believe that shallow cloning and some replication/caching on the GitHub side might be causing intermittent build failures.

Merged after a quick chat with @JonathanGiles


How to dismiss the snackbar

I notice there is no sample code to dismiss the snackbar. Wondering how to do it?

Late to the party, but if someone else stumbles here:
When you set it to visible: true you need to setTimeout.
this.setState({visible: true}, () => {
                        setTimeout(()=> {
                            this.setState({visible: false})
                        }, 3000)
                    })


fix: running queues after stop

resolves #321

The issue with this current approach is that the switch doesn't currently prohibit dials when it is stopped (which is why some tests are failing, they never start the switch). While libp2p currently prohibits dials when it's offline, it may be something that warrants changing in the future, https://github.com/libp2p/js-libp2p/issues/341.
It might be simpler to just ensure the target queue exists before adding it and starting it, to avoid the change in switch behavior. This is likely an edge case with running queues shutting down while the queues are draining.

...but if the switch is stopped isn't it reasonable for it to disallow dials? I take the point on https://github.com/libp2p/js-libp2p/issues/341 but should there instead be different modes of operation? e.g. "start in dial only mode"
My issue is that https://github.com/libp2p/js-libp2p-switch/pull/323 will guard against the edge case but it doesn't actually solve the issue of the node continuing to run after it has been "stopped".

My issue is that #323 will guard against the edge case but it doesn't actually solve the issue of the node continuing to run after it has been "stopped".

Yeah sorry, I confused the goal being to just fix #323. I am looking into an issue where there are connections staying open after close, even when the queues are stopped which is likely contributing to this problem.
I do think a configuration of dialOnly or offline is the best route to go for supporting libp2p/js-libp2p#341 properly.

Yeah sorry, I confused the goal being to just fix #323.

I should have made it clearer. Do you think what's here is valuable? If so, I'm happy to do the work the fix up the tests.

Do you think what's here is valuable?

Yes, what if we had the dial manager initialize with this._isRunning set to true instead of false? This would keep the current functionality of allowing offline dials, but once start/stop actions are taken it's no longer considered offline and the dial manager will behave as such.

Superceded by #324


make numa support in cmdq3 optional in cmake

The CMakeLists.txt for fpgadiag (and cmdq3) currently doesn't use a CMake
package finder to find numa and instead only looks for the library.
If it finds the library but the development files (headers) aren't present
on the system, the build can run into issues trying to include numa.h.
The PR makes it a CMake option to turn on numa support in cmdq3.
TODO: Use (and maybe develop) a numa package finder to check for numa.h


[Bug?]Unknown issue causing server freezes even in singleplayer world

After installing this mod and activating many features, entered a singleplayer world to do some testing.
After a while, I paused then unpaused, and all mobs ceased movement, and everything just stopped working.
Checking the MultiMC console, i recieved the following messages despite being on a singleplayer world:
[15:57:31] [Server thread/WARN]: Can't keep up! Is the server overloaded? Running 130486ms or 2609 ticks behind
[15:58:37] [Server thread/WARN]: Can't keep up! Is the server overloaded? Running 51161ms or 1023 ticks behind
[15:59:06] [Server thread/WARN]: Can't keep up! Is the server overloaded? Running 13959ms or 279 ticks behind
with no other error messages, unfortunately.
So something, after installing just this mod, caused even a singleplayer world to almost completely cease ticking.
 Unknown cause- bad mod interactions maybe?
I'll have to do more testing. In the meantime just thought I'd bring it to your attention in case you may have any ideas.
Singleplayer mod list for reference:
(Yes, I know Optifine is terrible, and Optifabric is unsupported, but unfortunately, I require it for most of the features of my resource packs to function, as there are no alternatives for many of the features. Also OF is clientside so it shouldn't be causing this if it's an interaction with the mod?... I think?)
accurate-block-placement-1.0.7.jar
appleskin-mc1.16-fabric-1.0.11.jar
borderless-mining-1.0.6+1.16.2.jar
cleancut-mc1.16-2.1-fabric.jar
clear-skies-mc116-1.5.48.jar
cleardespawn-1.16.2-fabric0.17.2-1.1.1.jar
CompactStatusEffects-1.2.2.jar
darkness-mc116-1.6.50.jar
DynamicSoundFilters-1.1.3+1.16.2.jar
EnchantedToolTips-1.3.4-1.16.jar
Fabric-1.16.3-TillItBreaks-0.1.jar
fabric-api-0.24.1+build.412-1.16.jar
fabric-language-kotlin-1.4.0+build.1.jar
fabrication-1.0.2.jar
horse-stats-vanilla-4.1.4.jar
i-am-very-smart-2.1.0+mc1.16.1.jar
illuminations-1.2.2.jar
light-overlay-5.5.3.jar
lock_minecart_view-for-MC1.16.2-1.2.1.jar
microDurability-0.2.0+1.16.3.jar
modlist.txt
modmenu-1.14.6+build.31.jar
mousewheelie-1.5.7+mc1.16.2-pre1.jar
nightvisionflashbegone-1.0.0.jar
nomoreglowingpots-1.0.2.jar
optifabric-1.6.4.jar
OptiFine_1.16.3_HD_U_G3.jar
phosphor-fabric-mc1.16.1-0.6.0+build.7.jar
potiondescriptions-1.16.3-fabric-1.0.0.jar
PresenceFootsteps-r28+1.16.2.jar
RoughlyEnoughItems-5.6.1.jar
slight-gui-modifications-1.4.0.jar
stepup-1.16.2-fabric0.17.2-1.1.jar
tooltipfix-1.0.2-1.16.jar
torohealth-1.16.3-fabric-1.jar
XaerosWorldMap_1.11.0_Fabric_1.16.3.jar
Xaeros_Minimap_20.26.0_Fabric_1.16.3.jar
[1.16.2]bettermodbutton-1.1.4.jar

Try disabling better_pause_freezing.

That does appear to have solved it, or at the very least I've been unable to reproduce the issue.

Yeah I figured making the game clock stop while paused was a Bad Idea™.
Will remove that.


App Services (API) - Deployment Slots and CORS

There might be a potential bug with deployment slots and CORS.
Preflight service calls from browser to the production app service (api) are getting routed to the deployment slot with > 0% traffic %, this will cause CORS error to your application.
I wanted to report this, but the "Feedback" button of App Services page leads to a broken link.
Happy to provide more detail to help others from running into the same issues.  If someone can redirect to the right page to report this, that'll be greatly appreciated.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: f6e09089-1ae2-8943-5ce2-9d48f458c81f
Version Independent ID: ba780cba-f604-b0a4-a81a-23c7d2384762
Content: Set up staging environments - Azure App Service
Content Source: articles/app-service/deploy-staging-slots.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

Hey @alaconda, the fact you're app is an API shouldn't be an issue. How are your AllowedOrigins configured? Did you inspect x-ms-routing-name in your header?

Hi @RyanHill-MSFT,
After spending more time investigating into this.  It turns out that the API in the deployment slot was not started properly and it was throwing 503.  I was thrown off by the CORS error messages, and not realized that the API is not working.  Sorry for the confusion.

No worries @alaconda, glad to hear it was sorted out.


move api docs to github pages

see https://github.com/DTLaboratory/dtlab-api-docs
and https://github.com/DTLaboratory/dtlab-ingest-api-docs


Why not to use cmake source code instead?

Hi. I'm surpised that you use rust instead of C++ to develop this LSP server. It's reasonable to use CMake source code base instead of trying to port its features to inherently incompatible language. Is there motivation behind this?

Languager server is just about ast,  search packages and complete , it is just analyze the text, so any languages are possible, and it is easy for rust to develop language server

It would be best IMHO to reuse existing codebase in order to avoid maintenance burden.

It would be best IMHO to reuse existing codebase in order to minimize maintenance burden.

This program use treesitter to handle cmake ast,  cmake repo is too big, and no one really like to develop in cpp


Changing mutable borrow check error when it occurs in a loop

I'm happy to write up a full RFC for this, but I'd prefer to float the idea and understand if more contextual messages are a good thing. Inspired by my own experience in rust-lang/rust#43437, the message for two mutable borrows doesn't account for the case of loops - producing an error message that looks strange and doesn't really communicate what the borrow checker is having a problem with.
I'd like to change it so that the formatted message is changed based on it occuring inside a loop, similar to the use of moved value error message, which specifies when the move occurs in a loop iteration.
The code is already written, in https://github.com/ivanbakel/rust/commit/53fb2a5bedae0633b33385972f6c1164585d9652 .

I think that a PR would be sufficient for this. In general, if we can detect a particular case accurately (and with little overhead for the non-erroring path) more specific error messages are greatly preferable.

What do you mean by a PR - a RFC PR, or a PR directly to the rust-lang/rust repo?

Sorry, a PR directly to rust-lang/rust.

My PR was merged, so I'm closing this.


fix(k8s): volumeMounts gets consistent volumeMixin arg

As mentioned on Slack recently ... the volumeMixin arg is useful to have on all of these functions.
@malcolmholmes

Dupe of https://github.com/jsonnet-libs/k8s/pull/183?

Dupe of #183?

You're right! didn't realize... I've had this override in-house for a very long time and only checked main branch here. Did we make them identical? ...


Remove Print after Select

Feature request
Is your feature request related to a problem? Please describe.
To keep my debug output clean
Describe the solution you'd like
Pls tell me how to disable this
Describe alternatives you've considered
I've looking through the supabase scripts, cant find which one is keeps on printing
Additional context
Add any other context or screenshots about the feature request here.

SORRY I WAS STUPID
turns out it was on an error from y ends
:)

No worries!


After converting “かっこ” to parentheses （） in the predictive conversion, pressing the Move button in the sentence returns the status before the predictive conversion.

Steps to reproduce

input かっこ in japanese
Select （） from the predictive conversion
Press the button to move the cursor forward.

Expected results
the text should be （） and the cursor should be inside the （）
Actual results
the text is かっこ
Code sample
Code sample
import 'package:flutter/material.dart';

class SamplePage extends StatefulWidget {

  SamplePage(): super();

  @override
  SamplePageState createState() => SamplePageState();
}

class SamplePageState extends State<SamplePage> with WidgetsBindingObserver {
  final TextEditingController _controller = TextEditingController();
  final FocusNode _focusNode = FocusNode();

  @override
  Widget build(BuildContext context) {
    return Scaffold(
      body: Center(
        child: TextFormField(
          controller: _controller,
          focusNode: _focusNode,
          onChanged: (String value) {
            setState(() { });
          },
        ),
      )
    );
  }
}

Screenshots or Video

Screenshots / Video demonstration
https://github.com/flutter/flutter/assets/7422251/cc5a8f8e-3b0b-4334-b3d6-171983f22923

Logs
Logs
[Paste your logs here]

Flutter Doctor output
Doctor output
[√] Flutter (Channel stable, 3.19.6, on Microsoft Windows [Version 10.0.22631.3593], locale ja-JP)
    • Flutter version 3.19.6 on channel stable at C:\Users\81708\Desktop\develop\flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 54e66469a9 (5 weeks ago), 2024-04-17 13:08:03 -0700
    • Engine revision c4cd48e186
    • Dart version 3.3.4
    • DevTools version 2.31.1

[√] Windows Version (Installed version of Windows is version 10 or higher)

[√] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at C:\Users\81708\AppData\Local\Android\sdk
    • Platform android-34, build-tools 31.0.0
    • Java binary at: C:\Program Files\Android\Android Studio1\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)
    • All Android licenses accepted.

[√] Chrome - develop for the web
    • Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe

[X] Visual Studio - develop Windows apps
    X Visual Studio not installed; this is necessary to develop Windows apps.
      Download at https://visualstudio.microsoft.com/downloads/.
      Please install the "Desktop development with C++" workload, including all of its default components

[√] Android Studio (version 2021.3)
    • Android Studio at C:\Program Files\Android\Android Studio1
    • Flutter plugin can be installed from:
       https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
       https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.13+0-b1751.21-8125866)

[√] VS Code (version 1.89.1)
    • VS Code at C:\Users\81708\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.88.0

[√] Connected device (4 available)
    • sdk gphone x86 (mobile) • emulator-5554 • android-x86    • Android 11 (API 30) (emulator)
    • Windows (desktop)       • windows       • windows-x64    • Microsoft Windows [Version 10.0.22631.3593]
    • Chrome (web)            • chrome        • web-javascript • Google Chrome 125.0.6422.60
    • Edge (web)              • edge          • web-javascript • Microsoft Edge 125.0.2535.51

[√] Network resources
    • All expected network resources are available.

! Doctor found issues in 1 category.

This is not a problem I found, but a problem my app users found
I could not find the same problem on my test device, but it may be a problem with a specific device.
Here is the user's device information
deviceName:SC-53B
osVersion:34

Also, as a funny behavior in this video, if you convert the “かっこ” to () in the first place, the cursor should be in the middle of the ()
I'm wondering if this is a similar problem, so I'll report it as well.

Hi @hatano0x06
I checked this on my device Pixel 7 Android 14 but couldn't see the issue on both Flutter 3.19.6 and 3.22.0. かっこ turns to () and the cursor is in the middle of () as desired.

Demo issue is not reproducible
https://github.com/flutter/flutter/assets/104349824/e875744e-dc97-47d5-bad6-bd9cbbf52c8a

Maybe this is a device-specific issue and it looks like a Samsung device (Galaxy A52 5G SC-53B). I found an issue at https://github.com/flutter/flutter/issues/120351 that is somewhat similar to this. Even though it's not exactly the same, I see it occurs on the same text かっこ and it doesn't turn to an expected text when adding a line break; and it happens on Samsung device too.
It could be the same cause, so I'm going to centralize this to that one. Please follow up on it and also leave your case there that may help to raise the issue. Thanks!


Improve errors in PreCloudInitIso method

What this PR does / why we need it: This MR improves the errors in the PreCloudInitIso method. Now we include errors when we fail on Marshal function. Also changed Infof logs to Error when error occurs.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged): None
Special notes for your reviewer: None
Release note:

NONE

/ok-to-test
/approve

/retest

/lgtm